ubiquibot / comment-incentives

0 stars 10 forks source link

Broken Permit Format #3

Closed 0x4007 closed 5 months ago

0x4007 commented 5 months ago

Please match the permit encoding format in this codebase.

ubiquibot[bot] commented 5 months ago

@whilefoo the deadline is at 2024-01-25T19:04:25.638Z

whilefoo commented 5 months ago

It appears the permit was not generated with new format. I think you haven't merged https://github.com/pavlovcik/ubiquibot-config/pull/7 to this repo

0x4007 commented 5 months ago

Perhaps you can open another pull request to @ubiquibot organization's config?

whilefoo commented 5 months ago

Won't we use this repo (comment-incentives) for this? It seems weird to have it in config repo

0x4007 commented 5 months ago

I can't load the pull request file changes view from my phone it just breaks so I don't understand the implementation details. Anyways it probably makes sense to just merge in this repository then!

whilefoo commented 5 months ago
image

I can't make a PR

0x4007 commented 5 months ago

That's weird what about trying to merge to development? I'm inclined to think user error cause git has been around for awhile.

whilefoo commented 5 months ago

I think it's because the two repositories don't have a common root commit (https://stackoverflow.com/a/23350970), so the only way is to force push or create new repo

0x4007 commented 5 months ago

Just commit on development and in the commit message you can add a new line

Resolves #3

Although a few days ago I merged unrelated histories by adding a flag. Something like

git merge --allow-unrelated-histories
ubiquibot[bot] commented 5 months ago
+ Evaluating results. Please wait...
ubiquibot[bot] commented 5 months ago

[ 38.6 WXDAI ]

@pavlovcik
Contributions Overview
ViewContributionCountReward
IssueSpecification112
IssueComment422.8
ReviewComment13.8
Conversation Incentives
CommentFormattingRelevanceReward
- https://github.com/ubiquity/pay.ubq.fi/pull/146 appears to hav...
12
li:
  count: 3
  score: "3"
  words: 24
112
Perhaps you can open another pull request to @ubiquibot organiza...
2.40.242.4
I can't load the pull request file changes view from my phone it...
7.20.267.2
That's weird what about trying to merge to development? I'm incl...
4.80.094.8
Just commit on development and in the commit message you can add...
8.4
code:
  count: 1
  score: "1"
  words: 0
0.338.4
Glad to see you figured it out. I'll try and review on my comput...
3.80.233.8

[ 62.3 WXDAI ]

@whilefoo
Contributions Overview
ViewContributionCountReward
IssueTask1.0050
IssueComment40
IssueComment411.3
ReviewComment10.5
ReviewComment10.5
Conversation Incentives
CommentFormattingRelevanceReward
It appears the permit was not generated with new format. I think...
-0.59-
Won't we use this repo (`comment-incentives`) for this? It seems...
-
code:
  count: 1
  score: "0"
  words: 2
0.16-
imageI think it's because the two repositories don't have a common ro...
-0.46-
It appears the permit was not generated with new format. I think...
2.70.592.7
Won't we use this repo (`comment-incentives`) for this? It seems...
2.9
code:
  count: 1
  score: "1"
  words: 2
0.162.9
imageI think it's because the two repositories don't have a common ro...
3.20.463.2
finally managed to do it ...
0.50.350.5
finally managed to do it ...
0.50.350.5
0x4007 commented 5 months ago

[ 38.6 WXDAI ]

@pavlovcik

[ 62.3 WXDAI ]

@whilefoo

Hey @rndquu @wannacfuture we need to fix the payment permit wallet because these rewards are coming from a non funded wallet. I want to make a new wallet and start issuing these payments again.

@wannacfuture looks like relevance is not multiplying against the formatting score. This is incorrect.

wannacfuture commented 5 months ago

You just need to change the X25519_PRIVATE_KEY and evmKey in the config.

0x4007 commented 5 months ago

I was never able to successfully regenerate the evm encrypted private key. I might need some help with this but I'll try again tonight.

@Diamondnegroni will keep you posted on if this is stable

rndquu commented 5 months ago

[ 38.6 WXDAI ]

@pavlovcik

[ 62.3 WXDAI ]

@whilefoo

Hey @rndquu @wannacfuture we need to fix the payment permit wallet because these rewards are coming from a non funded wallet. I want to make a new wallet and start issuing these payments again.

@wannacfuture looks like relevance is not multiplying against the formatting score. This is incorrect.

If you want to use a new wallet then you need to:

  1. Export evm private key of the new wallet
  2. Open https://pay.ubq.fi/keygen
  3. Insert x25519_private_key in the "X25519_PRIVATE_KEY" field (I've DMed you this value on 25 Dec, check telegram)
  4. Insert evm private key in the "PLAIN_TEXT" field
  5. Click "Encrypt"
  6. Insert the value from the "CIPHER_TEXT" field in the bot's config (not sure where exactly)