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

Allowlist: Add red x to error message #2055

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

Conversation

siljapetasch
Copy link

When user wants to add an address to the allowlist that already has a role, an error message is printed. This one is hard to see, so I suggest adding a red X to the message

@arturrez
Copy link
Collaborator

arturrez commented Jul 25, 2024

please merge latest main and resolve the conflict
once done I should be able to approve it @siljapetasch //cc @felipemadero

@siljapetasch
Copy link
Author

@arturrez merged recent changes as requested

@felipemadero
Copy link
Collaborator

@siljapetasch hi, thanks for the collaboration. we need all commits to be signed. once that is done, we will merge

@arturrez arturrez requested a review from a team as a code owner September 26, 2024 19:21
@arturrez arturrez self-requested a review September 26, 2024 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Backlog 🗄️
Development

Successfully merging this pull request may close these issues.

4 participants