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

Add workflow to check that core contracts storage is not corrupted #775

Closed
rndquu opened this issue Aug 25, 2023 · 25 comments · Fixed by #789
Closed

Add workflow to check that core contracts storage is not corrupted #775

rndquu opened this issue Aug 25, 2023 · 25 comments · Fixed by #789
Assignees
Labels

Comments

@rndquu
Copy link
Member

rndquu commented Aug 25, 2023

Depends on #774

Original idea

When we add UUPS support to the core contracts we should check that storage is not corrupted on each PR. There is a handy action for it.

What should be done

  • Add a new workflow that uses the rubilmax/foundry-storage-check action on each PR and push to the development branch when *.sol files have been updated
@gitcoindev
Copy link
Contributor

/start

@ubiquibot
Copy link

ubiquibot bot commented Sep 24, 2023

Task Creator Reward

rndquu: [ CLAIM 40.5 WXDAI ]

@ubiquibot
Copy link

ubiquibot bot commented Sep 24, 2023

@molecula451
Copy link
Member

molecula451 commented Sep 24, 2023

the bot didn't generate rewards for assignee here either @pavlovcik

@0x4007
Copy link
Member

0x4007 commented Sep 24, 2023

I assume that it is most likely to time out when it calculates the assignee rewards for some reason

@gitcoindev
Copy link
Contributor

Hi @pavlovcik @rndquu @molecula451 thank you for approving and merging! I see this ticket was not the only one affected, likely a regression from one of the recent bot pull requests. Is there any chance to manually trigger the bot to regenerate the assignee reward ?

@0x4007
Copy link
Member

0x4007 commented Sep 25, 2023

@0xcodercrane either you can push a hot fix or I'll need to manually send payment.

I only ask cause you messaged me today saying you're working on the hot fix.

@gitcoindev
Copy link
Contributor

Hi @pavlovcik is the hotfix available?

@0x4007
Copy link
Member

0x4007 commented Sep 28, 2023

@rndquu is working on it. We just need to open and close the issues again to generate the outstanding payments.

@rndquu rndquu reopened this Sep 28, 2023
@rndquu rndquu closed this as completed Sep 28, 2023
@ubiquibot
Copy link

ubiquibot bot commented Sep 28, 2023

@gitcoindev please be sure to review this conversation and implement any necessary fixes. Unless this is closed as completed, its payment of 40.5 WXDAI will be deducted from your next bounty.

@ubiquibot
Copy link

ubiquibot bot commented Sep 28, 2023

@rndquu rndquu reopened this Sep 28, 2023
@ubiquibot
Copy link

ubiquibot bot commented Sep 28, 2023

@gitcoindev please be sure to review this conversation and implement any necessary fixes. Unless this is closed as completed, its payment of 40.5 WXDAI will be deducted from your next bounty.

@rndquu rndquu closed this as completed Sep 28, 2023
@molecula451
Copy link
Member

most likely the bot already savePermit() the permit to DB, this happened at ubiquity/ubiquibot#797

@gitcoindev
Copy link
Contributor

I hope the bot did not save the penalty as well 😆 , it seems to go pretty rogue recently.

@molecula451
Copy link
Member

no, it does not, but it shows the penalty because it most likely already saved the permit

@gitcoindev
Copy link
Contributor

Ok, so when it saved the permit before it assumed all done and did not re-generate the assignee reward, just the conversation rewards, interesting...

@molecula451
Copy link
Member

No. It did not regenerate because it's already saved to the DB so thats why it Pops the penalty if it's reopen because the bot it's fetching the permit data what it did not was comment that permit here

@molecula451
Copy link
Member

molecula451 commented Sep 28, 2023

Also The bot is referencing a penalty amount based on the creator tasks award of first mention about the rewards not based on the task label i see this is happening in all others too You may see the task is 50 but the bot is saying a penalty of 40.5 referencing to creator awards in this case but not limited to

@0x4007
Copy link
Member

0x4007 commented Oct 4, 2023

/query @gitcoindev

@ubiquibot
Copy link

ubiquibot bot commented Oct 4, 2023

@gitcoindev's wallet address is 0x7e92476D69Ff1377a8b45176b1829C4A5566653a and multiplier is 1

@0x4007
Copy link
Member

0x4007 commented Oct 4, 2023

@gitcoindev
Copy link
Contributor

@pavlovcik thank you

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

Successfully merging a pull request may close this issue.

4 participants