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

Private discussions and Private tags grant money #32

Open
katosdev opened this issue Dec 31, 2022 · 3 comments · May be fixed by #44
Open

Private discussions and Private tags grant money #32

katosdev opened this issue Dec 31, 2022 · 3 comments · May be fixed by #44

Comments

@katosdev
Copy link

There should be a tag picker to allow administrators to select which tags count towards money earned for discussions.
Private discussions also shouldn't count towards earned money.

@katosdev
Copy link
Author

Proposed solution

  1. In https://github.com/AntoineFr/flarum-ext-money/blob/master/src/Listeners/GiveMoney.php add a -$money for if a discussion is marked as private.
  2. In https://github.com/AntoineFr/flarum-ext-money/blob/master/src/Listeners/GiveMoney.php add a check for if is_private = 1 then don't grant money on a new discussion / post.

Further, in the admin settings there should be a tag picker to allow an admin to select which tags count towards gained money.
In https://github.com/AntoineFr/flarum-ext-money/blob/master/js/src/admin/index.js

You could theoretically add a setting to https://github.com/AntoineFr/flarum-ext-money/blob/master/js/src/admin/index.js to allow an admin to choose whether private discussions should count towards the gained money.

@AntoineFr
Copy link
Owner

Hello @katosdev , thank you for your suggestions!

I will look at the the private discussion setting in the next weeks as it looks quite simple.
For the tag picker I'll add that to my todo list but I can't promise I will add it soon.

@AntoineFr AntoineFr linked a pull request Jun 8, 2024 that will close this issue
@AntoineFr
Copy link
Owner

Hello, I started a Pull request to add an option to disable giving money if a discussion or a post is private: #44 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants