Skip to content

Improve#3

Merged
Alkarex merged 9 commits into
FreshRSS:masterfrom
math-GH:improve
Nov 4, 2025
Merged

Improve#3
Alkarex merged 9 commits into
FreshRSS:masterfrom
math-GH:improve

Conversation

@math-GH

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

Copy link
Copy Markdown
Contributor
  • 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

Comment thread i18n/fr/ext.php
Comment thread i18n/fr/gen.php Outdated
@math-GH

math-GH commented Nov 2, 2025

Copy link
Copy Markdown
Contributor Author

@Alkarex Any idea how to proceed here?

Comment thread i18n/fr/gen.php Outdated
@Alkarex

Alkarex commented Nov 4, 2025

Copy link
Copy Markdown
Member

@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

Alkarex commented Nov 4, 2025

Copy link
Copy Markdown
Member

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