Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Replace @user everywhere #793

Closed
0x4007 opened this issue Sep 21, 2023 · 47 comments · Fixed by #826
Closed

Replace @user everywhere #793

0x4007 opened this issue Sep 21, 2023 · 47 comments · Fixed by #826

Comments

@0x4007
Copy link
Member

0x4007 commented Sep 21, 2023

@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 ubiquity/ubiquity-dollar#774 (comment)

@molecula451
Copy link
Member

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

@ubiquibot
Copy link

ubiquibot bot commented Sep 22, 2023

Skipping /query because it is disabled on this repo

@0x4007
Copy link
Member Author

0x4007 commented Sep 22, 2023

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

Possibly but this is a separate issue

@EtherealGlow
Copy link
Contributor

/start

@EtherealGlow
Copy link
Contributor

is the bot sleeping because of my merged pr 💀

@Sadaf-A
Copy link
Contributor

Sadaf-A commented Sep 22, 2023

/start

@0x4007
Copy link
Member Author

0x4007 commented Sep 23, 2023

/start

Looks like it's broken?

@0x4007
Copy link
Member Author

0x4007 commented Sep 23, 2023

/help

1 similar comment
@molecula451
Copy link
Member

/help

@molecula451
Copy link
Member

/query @pavlovcik

@molecula451
Copy link
Member

broke hard

@0x4007
Copy link
Member Author

0x4007 commented Sep 23, 2023

/help

@ubiquibot
Copy link

ubiquibot bot commented Sep 23, 2023

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
Copy link
Member Author

0x4007 commented Sep 23, 2023

4b4c8f8

Invalid config according to the logs

@molecula451
Copy link
Member

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

@molecula451
Copy link
Member

/help

@ubiquibot
Copy link

ubiquibot bot commented Sep 23, 2023

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
Copy link
Contributor

Sadaf-A commented Sep 23, 2023

/start

@ubiquibot
Copy link

ubiquibot bot commented Sep 23, 2023

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.

    @EtherealGlow
    Copy link
    Contributor

    Mannn

    @Sadaf-A
    Copy link
    Contributor

    Sadaf-A commented Sep 23, 2023

    Mannn

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

    @EtherealGlow
    Copy link
    Contributor

    Nah you can have this issue

    @Sadaf-A
    Copy link
    Contributor

    Sadaf-A commented Sep 23, 2023

    /query @pavlovcik

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 23, 2023

    Error retrieving access for @pavlovcik

    @Sadaf-A
    Copy link
    Contributor

    Sadaf-A commented Sep 23, 2023

    /query @Sadaf-A

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 23, 2023

    Error retrieving access for @Sadaf-A

    @0x4007
    Copy link
    Member Author

    0x4007 commented Sep 23, 2023

    /allow-time @Sadaf-A true

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 23, 2023

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

    @0x4007
    Copy link
    Member Author

    0x4007 commented Sep 23, 2023

    /allow set-time @Sadaf-A true

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 23, 2023

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

    @0x4007
    Copy link
    Member Author

    0x4007 commented Sep 23, 2023

    /query @Sadaf-A

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 23, 2023

    @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
    Copy link
    Member Author

    0x4007 commented Sep 23, 2023

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

    @Sadaf-A
    Copy link
    Contributor

    Sadaf-A commented Sep 23, 2023

    /query @Sadaf-A

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 23, 2023

    @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
    Copy link
    Contributor

    Sadaf-A commented Sep 23, 2023

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

    @0x4007
    Copy link
    Member Author

    0x4007 commented Sep 23, 2023

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

    @Sadaf-A
    Copy link
    Contributor

    Sadaf-A commented Sep 23, 2023

    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
    Copy link
    Member Author

    0x4007 commented Sep 23, 2023

    Yes

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 25, 2023

    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
    Copy link
    Member

    no conversation rewards expected? @pavlovcik

    @EtherealGlow
    Copy link
    Contributor

    Did you guys up the price mul

    @Sadaf-A
    Copy link
    Contributor

    Sadaf-A commented Sep 25, 2023

    /query @Sadaf-A

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 25, 2023

    @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
    Copy link
    Contributor

    Sadaf-A commented Sep 25, 2023

    @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
    Copy link
    Member Author

    0x4007 commented Sep 25, 2023

    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
    Copy link
    Member Author

    0x4007 commented Sep 25, 2023

    @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.

    Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
    Projects
    None yet
    Development

    Successfully merging a pull request may close this issue.

    4 participants