Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

English docs fixes #14

Merged
merged 11 commits into from
Nov 20, 2016
Merged

English docs fixes #14

merged 11 commits into from
Nov 20, 2016

Conversation

subic
Copy link
Contributor

@subic subic commented Nov 20, 2016

  • Some minor formatting fixes;

  • all the French documentation in /user added to English

  • updated English screenshots with correct text and latest UI version

  • added a sample Apache config to English version

  • added description of the Firefox integration to English version as described here.

  • added DS_Store to .gitignore


**Draft copy**

### Authentication with Persona
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Persona service has been stopped by Mozilla, so this section can be removed.

- We also need some information:
+ Your FreshRSS version (on about page or `constants.php` file)
+ Your server configuration: type of hosting, PHP version
+ Your storage system (MySQL / MariaDB or SQLite)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or PostgreSQL


### HTTP Authentication

1. Leaves nothing visible
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should update this section to explain that the .htaccess must be placed in the ./i/ directory, so that the API, PubsubHubbub, etc. are working.

@Alkarex
Copy link
Member

Alkarex commented Nov 20, 2016

This is excellent work, @subic , many thanks!
Let me know whether you can address some of the comments I have just made.

@subic
Copy link
Contributor Author

subic commented Nov 20, 2016

Thanks for the reviews, I have added the changes and also removed the Mozzila persona from the French docs.

@Alkarex Alkarex merged commit ea9a882 into FreshRSS:master Nov 20, 2016
@Alkarex
Copy link
Member

Alkarex commented Nov 20, 2016

Very good :-)

Alkarex added a commit to FreshRSS/FreshRSS that referenced this pull request Nov 20, 2016
@Alkarex
Copy link
Member

Alkarex commented Nov 20, 2016

@marienfressinaud Can you please pull the changes on http://doc.freshrss.org ?

@marienfressinaud
Copy link
Member

Done :) thanks a lot @subic!

javerous pushed a commit to javerous/FreshRSS that referenced this pull request Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants