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

ReadY for Review - Quadraticlands diplomacy (a medley of Aditya/Richard/Owocki Code) #9039

Merged
merged 73 commits into from
Jul 12, 2021

Conversation

owocki
Copy link
Contributor

@owocki owocki commented Jun 11, 2021

Description

Quadratic Diplomacy, a tool where useres can stake their GTC on each other and improve each others trust bonus by doing so.
v1 features:

  • create room
  • add user to room
  • vote on users
  • trollbox chat
  • vouch on other users.

for next version (hopefully by GR11):

  • ability to do quadratic (instead of 1:1) rooms
  • actual staking upon vote with slashing if u lie.
Refers/Fixes

#anti-sybil-staking in

Testing

tested locally and it works well!

PLEASE TEST ON A FRESH QUAD LANDS DB, THERE ARE SOME CHANGES WHERE IF U PREVIOUSLY PULLED + TRY TO TEST THOSE ROOMS IT WONT WORK

screencapture-localhost-8000-quadraticlands-mission-diplomacy-2021-07-02-08_39_02
screencapture-localhost-8000-quadraticlands-mission-diplomacy-54f30bc3-c038-4f6b-ac3d-57fe4e775677-owocki-and-frens-2021-07-02-08_38_36

app/assets/v2/js/wallet.js Outdated Show resolved Hide resolved
@owocki owocki changed the title WIP - Quadraticlands diplomacy - Owocki Edition - Database Backed Read for Review - Quadraticlands diplomacy (a medley of Aditya/Richard/Owocki Code) Jul 2, 2021
@owocki owocki marked this pull request as ready for review July 2, 2021 14:43
app/assets/v2/js/wallet.js Outdated Show resolved Hide resolved
@owocki owocki changed the title Read for Review - Quadraticlands diplomacy (a medley of Aditya/Richard/Owocki Code) ReadY for Review - Quadraticlands diplomacy (a medley of Aditya/Richard/Owocki Code) Jul 2, 2021
@gdixon
Copy link
Contributor

gdixon commented Jul 5, 2021

Hey @owocki @scco,

How should the trust verification be validated? At the minute, if you create a room then you are automatically verified? If we don't tighten that up will we not just end up with thousands of rooms and no sybil protection?

@thelostone-mc
Copy link
Member

I've restricted access to staff aaa170f until we check the flow in live and address the question from @gdixon

Copy link
Contributor

@gdixon gdixon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good to me with the staff flags in place 🙌

@@ -122,6 +122,7 @@ async function fetchAccountData(provider) {
// MetaMask does not give you all accounts, only the selected account
console.log('Got accounts', accounts);
selectedAccount = accounts[0] || provider.account;
// selectedAccount = '0x1402e01ba957e030200ec21e1417dc32c11e486b';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this be removed before we push it out?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i kept just for the sake of testing ! will remove it out once we test it out in live as well

@thelostone-mc thelostone-mc merged commit 3752d82 into master Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants