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

Refactor Config: Automatically Derive weight / ref #495

Closed
Tracked by #490
0x4007 opened this issue Jul 11, 2023 · 29 comments · Fixed by #627
Closed
Tracked by #490

Refactor Config: Automatically Derive weight / ref #495

0x4007 opened this issue Jul 11, 2023 · 29 comments · Fixed by #627

Comments

@0x4007
Copy link
Member

0x4007 commented Jul 11, 2023

  • Remove the weight property in the default config.
  • During runtime, when a bounty is being assistive priced by the bot, the bot should parse the number from the Priority label and use it in place of the weight
  • Any words should be ignored from this parser, except for Priority: of course, in order to understand that it is a priority label.
  • Reference: feat: default config #383 (comment)
@EtherealGlow
Copy link
Contributor

EtherealGlow commented Jul 28, 2023

/start

@ubiquibot
Copy link

ubiquibot bot commented Jul 28, 2023

Deadline Fri, 28 Jul 2023 15:50:30 GMT
Registered Wallet 0x3b07d616EC780b22148b190A07F3829A11B11042
Payment Multiplier 1.00
Multiplier Reason
Total Bounty 37.5 USD
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

    /unassign

    @ubiquibot
    Copy link

    ubiquibot bot commented Jul 28, 2023

    You have been unassigned from the bounty @EtherealGlow

    @kamaalsultan
    Copy link
    Contributor

    kamaalsultan commented Aug 16, 2023

    /start

    @ubiquibot
    Copy link

    ubiquibot bot commented Aug 16, 2023

    @ByteBallet The time limit for this bounty is on Wed, 16 Aug 2023 19:18:42 UTC

    @0x4007
    Copy link
    Member Author

    0x4007 commented Aug 16, 2023

    /assign

    We recently changed the command to /start @ByteBallet

    @kamaalsultan
    Copy link
    Contributor

    /assign

    We recently changed the command to /start @ByteBallet

    i forgot...

    @kamaalsultan
    Copy link
    Contributor

    kamaalsultan commented Aug 16, 2023

    I think there is a mistake in default weight of time-labels.

    "time-labels": [
    {
    "name": "Time: <1 Day",
    "weight": 1,
    "value": 86400
    },
    {
    "name": "Time: <1 Week",
    "weight": 2,
    "value": 604800
    },
    {
    "name": "Time: <2 Weeks",
    "weight": 3,
    "value": 1209600
    },
    {
    "name": "Time: <1 Month",
    "weight": 4,
    "value": 2592000
    }
    ]

    I believe weight of "Time: <1 Month" must be 5 as "Time: <3 weeks" should be 4.
    @pavlovcik rfc

    @0x4007
    Copy link
    Member Author

    0x4007 commented Aug 16, 2023

    I think I updated the time labels to lower values recently with addresses this problem. Check out @ubiquity/.GitHub/.GitHub/ubiquibot-config.json for the new values.
    I think don't worry too much about those values for now.

    @0x4007
    Copy link
    Member Author

    0x4007 commented Aug 26, 2023

    @wannacfuture @0xcodercrane any idea why this isnt generating a payment permit? Nothing shows up on the logs. I'm assuming it has something to do with it being tracked by #490. Perhaps the bot thinks this is a meta issue or something.

    @wannacfuture
    Copy link
    Contributor

    no, it won't identify as meta issue. it checks checkbox

    @0x4007
    Copy link
    Member Author

    0x4007 commented Aug 27, 2023

    /query @ByteBallet

    @0x4007
    Copy link
    Member Author

    0x4007 commented Aug 27, 2023

    lol the bot is totally bricked on this repo apparently. Not great

    @ubiquibot
    Copy link

    ubiquibot bot commented Aug 28, 2023

    Invalid syntax for query command
    usage /query @user

    @kamaalsultan
    Copy link
    Contributor

    /query @ByteBallet

    @ubiquibot
    Copy link

    ubiquibot bot commented Aug 28, 2023

    @ByteBallet's wallet address is 0xeFeF11577Db8f882F6Bf92F46008170Ba3602D40 and multiplier is 1

    @kamaalsultan
    Copy link
    Contributor

    kamaalsultan commented Aug 28, 2023

    lol the bot is totally bricked on this repo apparently. Not great

    It seems it's an issue related to database. #574

    @kamaalsultan
    Copy link
    Contributor

    /wallet 0x30fDdD8b3f3a4183734884b6d383da6bf4bf2cd1

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 1, 2023

    Updated the wallet address for @ByteBallet successfully! Your new address: 0x30fDdD8b3f3a4183734884b6d383da6bf4bf2cd1

    @kamaalsultan
    Copy link
    Contributor

    /query @ByteBallet

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 1, 2023

    @ByteBallet's wallet address is 0x30fDdD8b3f3a4183734884b6d383da6bf4bf2cd1 and multiplier is 1

    @EtherealGlow
    Copy link
    Contributor

    how 💀

    @kamaalsultan
    Copy link
    Contributor

    kamaalsultan commented Sep 1, 2023

    how 💀

    My GH user name was ByteBallet and the wallet is assigned to ByteBallet on db. I changed my name to byteballet and it seems no wallet address was registered to byteballet before #495 (comment)
    Of course this is only guess since I can't access the db lol.
    It must be addressed and all the repos, orgs, or users must be managed by their id not name.

    @wannacfuture
    Copy link
    Contributor

    wannacfuture commented Sep 1, 2023

    how 💀

    My GH user name was ByteBallet and the wallet is assigned to ByteBallet on db. I changed my name to byteballet and it seems no wallet address was registered to byteballet before #495 (comment) Of course this is only guess since I can't access the db lol. It must be addressed and all the repos, orgs, or users must be managed by their id not name.

    Well actually after you have changed your username, it won't be a problem.
    Because there is no wallet address registered for your new name so that when the issue is closed the bot will ask to set your wallet address for your new username.

    Does it make sense? @ByteBallet

    Anyhow, I agree with that idea - using id instead of name

    @kamaalsultan
    Copy link
    Contributor

    kamaalsultan commented Sep 1, 2023

    Yes, I got the point. Then need another issue for it?

    @0x4007 0x4007 reopened this Sep 4, 2023
    @0x4007 0x4007 closed this as completed Sep 4, 2023
    @ubiquibot ubiquibot bot added the Permitted label Sep 4, 2023
    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 4, 2023

    Task Assignee Reward

    [ CLAIM 37.5 WXDAI ]

    0x30fDdD8b3...bf4bf2cd1

    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!

    @ubiquibot
    Copy link

    ubiquibot bot commented Sep 4, 2023

    Task Creator Reward

    pavlovcik: [ CLAIM 53.6 WXDAI ]

    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