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

closed issue #869

Closed
gitcoindev opened this issue Jan 18, 2024 · 5 comments
Closed

closed issue #869

gitcoindev opened this issue Jan 18, 2024 · 5 comments
Assignees
@gitcoindev

Comments

@gitcoindev
Copy link
Contributor

gitcoindev commented Jan 18, 2024 •
edited by molecula451
Loading

No description provided.

@gitcoindev gitcoindev mentioned this issue Jan 18, 2024
Closed
9 tasks
@gitcoindev gitcoindev self-assigned this Jan 18, 2024
@gitcoindev
Copy link
Contributor Author

Let me tackle this one first.

@molecula451
Copy link
Member

but only submit the PR that will be reference to the issue on the audit judging repo

@gitcoindev gitcoindev mentioned this issue Jan 18, 2024
Closed
@0x4007
Copy link
Member

0x4007 commented Jan 19, 2024 •
edited
Loading

I thought we agreed to resolve the ones on the judging repository, as I understand that we are still deciding which ones are valid etc. Why are we making copies here? It can make things out of sync if issues are still being decided on.

I think we should only mirror the issues once they are closed as completed for archival purposes.

Related #874 (comment)

@ubiquibot UbiquiBot

This comment was marked as resolved.

Sign in to view
@molecula451 molecula451 changed the title Fix Macho Heather Mammoth - AccessControlFacet doesn't have ability to set admin for the role sherlock 4 Jan 19, 2024
@molecula451 molecula451 closed this as not planned Won't fix, can't repro, duplicate, stale Jan 19, 2024
@ubiquibot UbiquiBot

This comment was marked as resolved.

Sign in to view
@molecula451 molecula451 changed the title sherlock 4 closed issue Jan 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gitcoindev gitcoindev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix set role admin gitcoindev/ubiquity-dollar
3 participants
@0x4007 @molecula451 @gitcoindev