Skip to content

Conversation

@math-GH
Copy link
Contributor

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

  • fixed missed i18n title of the about page.
  • improved the readme.md
  • improved the about.phtml:
    • use i18n strings
    • text explains a bit more
  • changed the i18n text, so that the extension results are more visible:
    • Save buttons have now another text
    • about page menu entry has now another text
  • i18n German added

@math-GH
Copy link
Contributor Author

math-GH commented Nov 2, 2025

@Alkarex Any idea how to proceed here?

@Alkarex
Copy link
Member

Alkarex commented Nov 4, 2025

@Alkarex Any idea how to proceed here?

I had completely forgotten about that repo. Let's merge this PR, and as following work, we need to add automated tests (e.g. PHPStan) to make it a good example for other extensions.

@Alkarex Alkarex merged commit df65958 into FreshRSS:master Nov 4, 2025
@Alkarex
Copy link
Member

Alkarex commented Nov 4, 2025

The logic of the extension should be reconsidered, to provide some good examples of hook usage, and limit overrides of existing files.

@math-GH math-GH deleted the improve branch November 4, 2025 20:06
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