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

Improve about page #5192

Merged
merged 10 commits into from
Mar 12, 2023
Merged

Improve about page #5192

merged 10 commits into from
Mar 12, 2023

Conversation

math-GH
Copy link
Contributor

@math-GH math-GH commented Mar 11, 2023

After:
grafik

Changes proposed in this pull request:

  • added: link to the documentation
  • added: external links will open in new tab
  • added: information about environment in data/config.php (shown for admins,Shown only, when it is not production. I thank that it is important additionally to the version number. )

How to test the feature manually:

  1. go to config -> about page
  2. see the page

check as guest user (not logged in) too:

  1. click on about FreshRSS on the login screen
  2. see the page

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

Additional information can be found in the documentation.

@math-GH math-GH added this to the 1.22.0 milestone Mar 11, 2023
app/i18n/fr/index.php Outdated Show resolved Hide resolved
app/i18n/en/index.php Outdated Show resolved Hide resolved
math-GH and others added 2 commits March 12, 2023 22:42
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
Co-authored-by: Alexandre Alapetite <alexandre@alapetite.fr>
@Alkarex Alkarex merged commit ce5572d into FreshRSS:edge Mar 12, 2023
@math-GH math-GH deleted the improve-about-page branch March 13, 2023 17:48
@math-GH math-GH restored the improve-about-page branch March 13, 2023 17:48
@math-GH math-GH deleted the improve-about-page branch March 13, 2023 17:49
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.

2 participants