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.

Sign up for GitHub

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

Jump to bottom

[ Feat ]: Revoke rights for a certain user #84

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[ Feat ]: Revoke rights for a certain user #84

wants to merge 4 commits into from

Conversation

brf153
Copy link
Contributor

@brf153 brf153 commented Jan 28, 2024

closes #79

Steps I have taken:

  1. Created a new command to revoke user rights.
  2. Implemented the /whiteboard deny {userName} of {boardName} command for this purpose.
  3. Extracted userName and boardName from the context parameters.
  4. Developed a new function to remove the specified userName from the boardOwner of the given board.
  5. Updated the board using the storeBoardData function.
  6. Sent a message to the board owner, notifying them that {userName} has been removed from the rights of {boardName}.

@brf153
feat:revoke user from board access
ded323d 
Signed-off-by: brf153 <153hsb@gmail.com>
@brf153
code cleaned
c8b16f5 
Signed-off-by: brf153 <153hsb@gmail.com>
@brf153
updated the readme
f90ea51 
Signed-off-by: brf153 <153hsb@gmail.com>
@brf153
Copy link
Contributor Author

brf153 commented Jan 28, 2024

whatsapp-video-2024-01-28-at-182707_cCSvGOTQ.mp4

@brf153 brf153 changed the title [Feat]: Revoke rights for a certain user [ Feat ]: Revoke rights for a certain user Jan 28, 2024
@brf153
code cleaned
01168a1 
Signed-off-by: brf153 <153hsb@gmail.com>
@CulturalProfessor
Copy link
Collaborator

hey @brf153 I think after your recent pr of permission I think we are not able to modify board name etc , please look into it

@brf153
Copy link
Contributor Author

brf153 commented Jan 29, 2024

@CulturalProfessor its working in my system

whatsapp-video-2024-01-29-at-230642_bpbsOAcs.mp4

@brf153
Copy link
Contributor Author

brf153 commented Jan 29, 2024

WhatsApp.Video.2024-01-29.at.23.14.21.mp4

@CulturalProfessor
Copy link
Collaborator

can you try after taking a pull it might be due to @Giridharan002 's pr

@brf153
Copy link
Contributor Author

brf153 commented Jan 29, 2024

@CulturalProfessor oki I will check it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[ Feat ] : Revoke board editing permissions for a user.
2 participants
@brf153 @CulturalProfessor