ubiquity / ubiquibot

Putting the 'A' in 'DAO'
https://github.com/marketplace/ubiquibot
MIT License
17 stars 59 forks source link

Replace @user everywhere #793

Closed 0x4007 closed 11 months ago

0x4007 commented 11 months ago

@user is supposed to be referencing a specific user usually who invoked the bot. I've seen this in a few places already and it must be fixed. Especially see it all the time on the /start table

Invalid syntax for query command usage /query @user

Originally posted by @ubiquibot in https://github.com/ubiquity/ubiquity-dollar/issues/774#issuecomment-1730056879

molecula451 commented 11 months ago

/query user (not called @) should still fetch "user"

ubiquibot[bot] commented 11 months ago

Skipping /query because it is disabled on this repo

0x4007 commented 11 months ago

/query user (not called @) should still fetch "user"

Possibly but this is a separate issue

EtherealGlow commented 11 months ago

/start

EtherealGlow commented 11 months ago

is the bot sleeping because of my merged pr 💀

Sadaf-A commented 11 months ago

/start

0x4007 commented 11 months ago

/start

Looks like it's broken?

0x4007 commented 11 months ago

/help

molecula451 commented 11 months ago

/help

molecula451 commented 11 months ago

/query @pavlovcik

molecula451 commented 11 months ago

broke hard

0x4007 commented 11 months ago

/help

ubiquibot[bot] commented 11 months ago

Available commands


- /start: Assign the origin sender to the issue automatically.
- /stop: Unassign the origin sender from the issue automatically.
- /help: List all available commands.
- /autopay: Toggle automatic payment for the completion of the current issue.
- /query: Comments the users multiplier and address
- /ask: Ask a technical question to the Ubiquity AI. 
  example usage: "/ask How do I do X?"
- /multiplier: Set the bounty payout multiplier for a specific contributor, and provide the reason for why. 
  example usage: "/wallet @user 0.5 'Multiplier reason'"
- /allow: Set access control. (Admin Only)
- /wallet: <WALLET_ADDRESS | ENS_NAME>: Register the hunter's wallet address. 
  ex1: /wallet 0x0000000000000000000000000000000000000000
  ex2: /wallet vitalik.eth
0x4007 commented 11 months ago

https://github.com/ubiquity/ubiquibot/commit/4b4c8f89909424f1c4ca37ebd16b513dae42816b

Invalid config according to the logs

molecula451 commented 11 months ago

but why commented out? did not tried to set to false instead?

molecula451 commented 11 months ago

/help

ubiquibot[bot] commented 11 months ago

Available commands


- /start: Assign the origin sender to the issue automatically.
- /stop: Unassign the origin sender from the issue automatically.
- /help: List all available commands.
- /autopay: Toggle automatic payment for the completion of the current issue.
- /query: Comments the users multiplier and address
- /ask: Ask a technical question to the Ubiquity AI. 
  example usage: "/ask How do I do X?"
- /multiplier: Set the bounty payout multiplier for a specific contributor, and provide the reason for why. 
  example usage: "/wallet @user 0.5 'Multiplier reason'"
- /allow: Set access control. (Admin Only)
- /wallet: <WALLET_ADDRESS | ENS_NAME>: Register the hunter's wallet address. 
  ex1: /wallet 0x0000000000000000000000000000000000000000
  ex2: /wallet vitalik.eth
Sadaf-A commented 11 months ago

/start

ubiquibot[bot] commented 11 months ago

Deadline Sat, 23 Sep 2023 03:15:48 UTC
Registered Wallet 0x0BEd00438D57d07E3667b85Fa8EB86Af147C7025
Tips:
EtherealGlow commented 11 months ago

Mannn

Sadaf-A commented 11 months ago

Mannn

shall I unassign myself if you want to work on it?

EtherealGlow commented 11 months ago

Nah you can have this issue

Sadaf-A commented 11 months ago

/query @pavlovcik

ubiquibot[bot] commented 11 months ago

Error retrieving access for @pavlovcik

Sadaf-A commented 11 months ago

/query @Sadaf-A

ubiquibot[bot] commented 11 months ago

Error retrieving access for @Sadaf-A

0x4007 commented 11 months ago

/allow-time @Sadaf-A true

ubiquibot[bot] commented 11 months ago

Invalid syntax for allow usage: '/allow set-(access type) @user true|false' ex-1 /allow set-multiplier @user false

0x4007 commented 11 months ago

/allow set-time @Sadaf-A true

ubiquibot[bot] commented 11 months ago

Updated access for @Sadaf-A successfully! Access: time for "ubiquity/ubiquibot"

0x4007 commented 11 months ago

/query @Sadaf-A

ubiquibot[bot] commented 11 months ago

@Sadaf-A's wallet address is 0x0BEd00438D57d07E3667b85Fa8EB86Af147C7025, multiplier is 1 and access levels are

access type access level
multiplier false
priority false
time true
price false
0x4007 commented 11 months ago

There is a bug @Sadaf-A but now you're good to query yourself.

Sadaf-A commented 11 months ago

/query @Sadaf-A

ubiquibot[bot] commented 11 months ago

@Sadaf-A's wallet address is 0x0BEd00438D57d07E3667b85Fa8EB86Af147C7025, multiplier is 1 and access levels are

access type access level
multiplier false
priority false
time true
price false
Sadaf-A commented 11 months ago

@pavlovcik can you elaborate a little more on what this issue is supposed to fix?

0x4007 commented 11 months ago

Any instance you see the string literal "@user" should never actually write "@user" in any bot comments.

Sadaf-A commented 11 months ago

Deadline Sat, 23 Sep 2023 03:15:48 UTC
Registered Wallet 0x0BEd00438D57d07E3667b85Fa8EB86Af147C7025
Tips:
  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address @user.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the bounty.

@pavlovcik I see like here the bot writes '@user' but what should it write instead? the actual user's tag?

0x4007 commented 11 months ago

Yes

ubiquibot[bot] commented 11 months ago

Task Assignee Reward

[ CLAIM 37.5 WXDAI ]

0x0BEd00438...f147C7025

If you've enjoyed your experience in the DevPool, we'd appreciate your support. Follow Ubiquity on GitHub and star this repo. Your endorsement means the world to us and helps us grow!
We are excited to announce that the DevPool and UbiquiBot are now available to partners! Our ideal collaborators are globally distributed crypto-native organizations, who actively work on open source on GitHub, and excel in research & development. If you can introduce us to the repository maintainers in these types of companies, we have a special bonus in store for you!

molecula451 commented 11 months ago

no conversation rewards expected? @pavlovcik

EtherealGlow commented 11 months ago

Did you guys up the price mul

Sadaf-A commented 11 months ago

/query @Sadaf-A

ubiquibot[bot] commented 11 months ago

@Sadaf-A's wallet address is 0x0BEd00438D57d07E3667b85Fa8EB86Af147C7025, multiplier is 1 and access levels are

access type access level
multiplier false
priority false
time true
price false
Sadaf-A commented 11 months ago

@pavlovcik the issue price is 25USD why did I receive 37.5? and even more confusing when I opened it. It was for 18.75 okay I get it. The bot deducted it because of the parse-permit-url PR

0x4007 commented 11 months ago

no conversation rewards expected? @pavlovcik

The bot is pretty broken. Fortunately due to the permit issuance design, in the future when it's stable we should be able to force regen on these old issues to generate the rewards.

0x4007 commented 11 months ago

@pavlovcik the issue price is 25USD why did I receive 37.5? and even more confusing when I opened it. It was for 18.75 okay I get it. The bot deducted it because of the parse-permit-url PR

I filed a new issue to fix this. You should be able to see the back reference on this conversation from a few hours ago.