#9 Add articles content in feed (new PR)

Merged
ohnonot merged 2 commits from raspbeguy/feed-content into ohnonot/master 4 years ago

I removed dependency to the feed/content variable and I explained this variable in the default config.

I removed dependency to the `feed`/`content` variable and I explained this variable in the default config.
ohnonot commented 4 years ago
Owner

This is the second time I'm asking you to also add documentation. It is just as important as the code itself. Please. The README is the documentation.

This is the second time I'm asking you to also add documentation. It is just as important as the code itself. Please. The README is the documentation.
raspbeguy commented 4 years ago
Poster

There is no documentation about the existing configuration. I thought you meant adding comments in the default config file.

EDIT: the readme doesn't include doc about feed config

There is no documentation about the existing configuration. I thought you meant adding comments in the default config file. EDIT: the readme doesn't include doc about feed config
ohnonot commented 4 years ago
Owner

the readme doesn't include doc about feed config

I can see that now. Sorry if I'm asking too much, but maybe you could add a few words? Thanks.

The new author page is also completely undocumented so far, see the issues I opened.

>the readme doesn't include doc about feed config I can see that now. Sorry if I'm asking too much, but maybe you could add a few words? Thanks. The new author page is also completely undocumented so far, see the issues I opened.
raspbeguy commented 4 years ago
Poster

Yes I'll do that this week, no problem.

Yes I'll do that this week, no problem.
ohnonot commented 4 years ago
Owner

Great. No hurries.

Great. No hurries.
This pull request has been merged successfully!
Sign in to join this conversation.
No Label
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.