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

feedback #2

Open
luceos opened this issue Nov 7, 2024 · 1 comment
Open

feedback #2

luceos opened this issue Nov 7, 2024 · 1 comment

Comments

@luceos
Copy link

luceos commented Nov 7, 2024

Hello @daocatt ,

I left some feedback about this extension on the Flarum official community: https://discuss.flarum.org/d/36248-translate-gtdxyzflarum-ext-badges/2

Here's a copy:

Some feedback for you:

  • instead of forking the original extension and publishing it under a new name, why did you not improve on the original one and make that one better, aside from the confusion you prevent with users, you wouldn't have had the issue with translations
  • although you forked the code from v17, you did not include their copyright information as is required by the same license you included; open source allows you to use code however you want, yet copyrights must remain intact

I applaud any effort to work in open source, make sure you do it respectfully though.
Good luck!

@daocatt
Copy link
Owner

daocatt commented Nov 8, 2024

Thanks for reply.

  • Some of the settings and features I need are very private ideas, so I don't think it's appropriate to merge them into the original plugin.
  • I will add source code information later.

Thanks again.

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

No branches or pull requests

2 participants