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

Bot should give a warning when conflicting tags are used #367

Open
Draeieg opened this issue Jun 5, 2023 · 2 comments · May be fixed by #878
Open

Bot should give a warning when conflicting tags are used #367

Draeieg opened this issue Jun 5, 2023 · 2 comments · May be fixed by #878
Labels
Price: 25 USD Priority: 2 (Medium) Time: <1 Hour

Comments

@Draeieg
Copy link
Member

Draeieg commented Jun 5, 2023

currently you can place 2 labels of the same category and the bot will assign tags anyway, not sure whats the logic for how it does the
price calculation

image

Bot should instead give am error/warning about conflicting tags in case an admin fat fingers the tags
@0x4007
Copy link
Member

0x4007 commented Jun 5, 2023

It uses the lowest/cheapest of the options. I think that the lowest/cheapest is a fine strategy but perhaps we can discuss why its a bad idea.

@ubiquibot ubiquibot bot mentioned this issue Jun 5, 2023
Closed
@Draeieg
Copy link
Member Author

Draeieg commented Jun 7, 2023

It uses the lowest/cheapest of the options. I think that the lowest/cheapest is a fine strategy but perhaps we can discuss why its a bad idea.

this logic seems fine my only concern is in situations like team need to give more priority to an issue but fat fingers it and don't remove the old tag, causing the bounty to not have updated pricing and be solved quicker

or a hunter reading two time labels and assuming that bounty can take up to that amount but the pay is only minimum

essentially it seems like a similar logic to #342

@harjaapdhillon16 harjaapdhillon16 mentioned this issue Jun 11, 2023
Open
@0x4007 0x4007 added ping and removed ping labels Aug 9, 2023
@0x4007 0x4007 removed the ping label Aug 29, 2023
@Keyrxng Keyrxng linked a pull request Oct 26, 2023 that will close this issue
Open
@korrrba-bot korrrba-bot bot mentioned this issue Nov 6, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Price: 25 USD Priority: 2 (Medium) Time: <1 Hour
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Tag warning Keyrxng/ubiquibot
2 participants
@0x4007 @Draeieg