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

feat: Restrict Comments on Posts in Groups #5599

Merged
merged 1 commit into from
Oct 27, 2022
Merged

Conversation

Mogge
Copy link
Contributor

@Mogge Mogge commented Oct 27, 2022

🍰 Pullrequest

Ensure that no posts in groups can be commented by users that are not member of the group.

Issues

Todo

  • None

@Mogge Mogge self-assigned this Oct 27, 2022
Copy link
Member

@Tirokk Tirokk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @Mogge
Very nice that you did this !!! 😍

What I have found is:

  • with this in the webapp I can comment on a post of a public group, but get an error from the backend if I submit the comment

In my eyes:

  • If commenting a post in a public group shall be possible …
    • we have to adjust the backend
  • If commenting a post in a public group shall not be possible …
    • we have to disable the commenting funktionallity on the post
    • and should display the reason why this is not possible
    • making a followup issue?

@Mogge Mogge merged commit 5efb530 into master Oct 27, 2022
@Mogge Mogge deleted the comments-on-posts-in-groups branch October 27, 2022 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants