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

Improved: Install process: give more infos #5350

Merged
merged 5 commits into from May 11, 2023

Conversation

math-GH
Copy link
Contributor

@math-GH math-GH commented Apr 27, 2023

Before:
No information about FreshRSS and the installing version
grafik

After:
Display some information
grafik

How to test the feature manually:

  1. Start the install process
  2. see the first step

Pull request checklist:

  • clear commit messages
  • code manually tested

@math-GH math-GH added UI 🎨 User Interfaces UX User experience labels Apr 27, 2023
@math-GH math-GH added this to the 1.22.0 milestone Apr 27, 2023
I do not find that informative, and quite confusing. Moved to readme instead.
@Alkarex
Copy link
Member

Alkarex commented May 9, 2023

I have followed-up #4450 in 50fcbae because there were still references to Kriss Feed and Leed in most languages, which were quite confusing, especially with this PR showing this info in the install page.

Please double-check e.g. for German.

@Alkarex Alkarex merged commit deb306c into FreshRSS:edge May 11, 2023
1 check passed
@math-GH math-GH deleted the install-give-more-infos branch May 11, 2023 20:04
math-GH added a commit to math-GH/FreshRSS that referenced this pull request Jul 4, 2023
* comments added for each step

* infos about FreshRSS added in first step

* Remove reference to Kriss and Leed from install page
I do not find that informative, and quite confusing. Moved to readme instead.

---------

Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI 🎨 User Interfaces UX User experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants