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

Broken Permit Format #3

Closed
0x4007 opened this issue Jan 25, 2024 · 15 comments · Fixed by #4
Closed

Broken Permit Format #3

0x4007 opened this issue Jan 25, 2024 · 15 comments · Fixed by #4

Comments

@0x4007
Copy link
Contributor

0x4007 commented Jan 25, 2024

Please match the permit encoding format in this codebase.

Copy link

ubiquibot bot commented Jan 25, 2024

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

@whilefoo
Copy link
Contributor

It appears the permit was not generated with new format. I think you haven't merged 0x4007/.ubiquity-os#7 to this repo

@0x4007
Copy link
Contributor Author

0x4007 commented Jan 25, 2024

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

@whilefoo
Copy link
Contributor

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

@0x4007
Copy link
Contributor Author

0x4007 commented Jan 25, 2024

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

image I can't make a PR

@0x4007
Copy link
Contributor Author

0x4007 commented Jan 26, 2024

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

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

0x4007 commented Jan 27, 2024

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

Copy link

ubiquibot bot commented Jan 30, 2024

+ Evaluating results. Please wait...

Copy link

ubiquibot bot commented Jan 30, 2024

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

0x4007 commented Jan 30, 2024

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

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

@0x4007
Copy link
Contributor Author

0x4007 commented Jan 31, 2024

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

rndquu commented Jan 31, 2024

[ 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)

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