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

Room-settings: Reject setting modjoin past % in groupchats #9484

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

Dirain1700
Copy link
Contributor

I could set modjoin to @ by using /modjoin sync, maybe this should be blocked?

@Dirain1700
Copy link
Contributor Author

IMG_6686
Also this should not be null

@Dirain1700 Dirain1700 marked this pull request as draft March 30, 2023 16:58
(actually i used 4spaces indent, so used GitHub web editor :p)
i havent tested yet, i will tomorrow
@Dirain1700 Dirain1700 marked this pull request as ready for review March 31, 2023 17:13
@Dirain1700
Copy link
Contributor Author

To be honest, I don't think it need to ban setting modjoin to trusted/whitelist, do you have any reasons to keep it this?

@mia-pi-git mia-pi-git changed the title fix(modjoin): Reject setting modjoin past % Room-settings: Reject setting modjoin past % Apr 1, 2023
@Dirain1700 Dirain1700 changed the title Room-settings: Reject setting modjoin past % Room-settings: Reject setting modjoin past % in groupchats Apr 2, 2023
@Dirain1700
Copy link
Contributor Author

image

@AnnikaCodes
Copy link
Collaborator

Global staff should be able to join groupchats yes. What happens if you set modchat to @ and then turn on /modjoin sync? Should there be a check in the modjoin command also?

@Dirain1700
Copy link
Contributor Author

Yes, currently it does so should check in modchat command.
IMG_7246

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