Skip to content

Task Done: Allow the user to make the bug report anonymous after posting (remove user from the issue). #1722

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
Merged
merged 4 commits into from
Jan 29, 2024
Merged

Task Done: Allow the user to make the bug report anonymous after posting (remove user from the issue). #1722

merged 4 commits into from
Jan 29, 2024

Conversation

@kr-2003
Copy link
Contributor

@kr-2003 kr-2003 commented Jan 29, 2024

Fixes #740

  • Created a method within the Issue model to handle remove user and call issue_instance.remove_user().
  • In remove_user, we can assign self.user = None.
  • Also made sure that the user performing the action is the user on the issue or the superuser.
  • Added respective routes.
Screen.Recording.2024-01-29.at.10.07.16.AM.mov

@kr-2003
Copy link
Contributor Author

kr-2003 commented Jan 29, 2024

@DonnieBLT Kindly review this PR.

DonnieBLT
DonnieBLT requested changes Jan 29, 2024
View reviewed changes
Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

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

Looks good, two requests. Thanks!

github-advanced-security[bot]
github-advanced-security bot found potential problems Jan 29, 2024
View reviewed changes
@kr-2003
Copy link
Contributor Author

kr-2003 commented Jan 29, 2024

Is this right way of redirection to different page? @DonnieBLT

@DonnieBLT DonnieBLT enabled auto-merge (squash) January 29, 2024 09:28
@DonnieBLT DonnieBLT merged commit 7fc039c into OWASP-BLT:main Jan 29, 2024
@kr-2003 kr-2003 mentioned this pull request Jan 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DonnieBLT DonnieBLT DonnieBLT approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Task: Allow the user to make the bug report anonymous after posting (remove user from the issue)

2 participants

@kr-2003 @DonnieBLT