Create a Newspaper builder. #8

New Issue

koalp · 2021-04-24T15:20:37+02:00

koalp commented

2021-04-24 15:20:37 +02:00

A Newspaper builder would be in charge of loging in with

enum Login {
    Username(String, String),
    Cookie(Vec<Cookie>),

}

as well as configuring the Newspaper ¹

For now, it is done directly by instaciating the structures, but it should be replaced by a NewspaperBuilder.

The newspaper builder could be an associated type of Newspaper.

the configuration structure should be generic so that it can be be retrieved from configuration files. ¿ How to do so ? With a trait ? It may require another issue ↩︎

A `Newspaper` builder would be in charge of loging in with ```rust enum Login { Username(String, String), Cookie(Vec<Cookie>), } ``` as well as configuring the `Newspaper` [^configuring] For now, it is done directly by instaciating the structures, but it should be replaced by a `NewspaperBuilder`. The newspaper builder could be an associated type of `Newspaper`. [^configuring]: the configuration structure should be generic so that it can be be retrieved from configuration files. ¿ How to do so ? With a trait ? It may require another issue

koalp added the

labels 2021-04-24 15:20:37 +02:00

koalp added this to the v0.1.0 - multiple newspaper, html only, chatbot milestone 2021-04-24 15:26:41 +02:00

koalp closed this issue

2021-04-24 16:01:42 +02:00

koalp reopened this issue

2021-04-24 16:01:45 +02:00

koalp added this to the 2021-04-26 - 2021-05-09 : sprint 2 project 2021-04-27 03:15:45 +02:00

koalp commented

2021-04-28 20:57:15 +02:00

The builder doesn't have to be generic for all Newspapers yet as there is no use for it yet.
A usecase would be dynamicly adding newspapers from config, but this will be reviewed later.

However, having tools to generate the builder and the implementation of Newspaper would be nice. This will be moved to #18

The builder doesn't have to be generic for all Newspapers yet as there is no use for it yet. A usecase would be dynamicly adding newspapers from config, but this will be reviewed later. However, having tools to generate the builder and the implementation of `Newspaper` would be nice. This will be moved to #18

koalp referenced a pull request that will close this issue

2021-04-29 02:04:10 +02:00

feat: add basic chatbot #14

koalp closed this issue

2021-04-29 02:24:20 +02:00

Sign in to join this conversation.