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

[1976] New description for direct voter registration #2005

Merged

Conversation

Ryun1
Copy link
Member

@Ryun1 Ryun1 commented Sep 10, 2024

List of changes

  • Change the direct voter registration message;
    • fix incorrect deposit info
    • generally improve clarity and distinction from DReps

Checklist

  • related issue
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the changelog
  • I have added tests that prove my fix is effective or that my feature works

@Ryun1 Ryun1 changed the title sketch new wording for direct voter registration [1976] Sketch new wording for direct voter registration Sep 10, 2024
Copy link

✅ All checks succeeded

Copy link

❌ Docker Image Scan (Dockle) failed

No output

@Ryun1
Copy link
Member Author

Ryun1 commented Sep 10, 2024

Thoughts @HeptaSean @disassembler and Johnny

Copy link

✅ All checks succeeded

@HeptaSean
Copy link

Thoughts @HeptaSean @disassembler and Johnny

Yes, like it. Maybe worth a mention that direct voters can, of course, delegate their other wallets and accounts to the “direct voter” dRep (and that makes a lot of sense, since they surely don't want to pay the deposit multiple times).

Also solves #1999 (that was mine) and #1976.

@HeptaSean
Copy link

As mentioned in #2001 (comment), I would still question the “direct voter” wording more fundamentally and maybe use “dRep with/without public information” and/or “(not) publically listed dRep” instead, but that should probably be discussed with more people and more time.

So, please do this first. It already makes it much better.

@HeptaSean
Copy link

Maybe worth a mention that direct voters can, of course, delegate their other wallets and accounts to the “direct voter” dRep

Just realised that that is not possible in gov.tools since you cannot give an arbitrary dRep ID to delegate to, but only choose from the list of (public) dReps. It is possible with Eternl and, of course, with CLI. So, that would also have to be in the text if you decide to add it.

Since it really makes a lot of sense that “direct voters” only use one dRep for all of their wallets and accounts, delegation to arbitrary dRep IDs should probably become a feature request (for the non-Eternl/CLI users).

@Ryun1
Copy link
Member Author

Ryun1 commented Sep 11, 2024

As mentioned in #2001 (comment), I would still question the “direct voter” wording more fundamentally and maybe use “dRep with/without public information” and/or “(not) publically listed dRep” instead, but that should probably be discussed with more people and more time.

Yup lets discuss the naming via #2012

Since it really makes a lot of sense that “direct voters” only use one dRep for all of their wallets and accounts, delegation to arbitrary dRep IDs should probably become a feature request (for the non-Eternl/CLI users).

Do you think this is a big use case?
We can look at supporting it properly

CC @m-i-k-e

jdyczka
jdyczka previously approved these changes Sep 11, 2024
Copy link

✅ All checks succeeded

Copy link

❌ Docker Image Scan (Dockle) failed

No output

Copy link

✅ All checks succeeded

1 similar comment
Copy link

✅ All checks succeeded

Copy link

❌ Docker Image Scan (Dockle) failed

No output

Copy link

✅ All checks succeeded

@jdyczka jdyczka dismissed their stale review September 11, 2024 13:47

approved too soon

@Ryun1
Copy link
Member Author

Ryun1 commented Sep 11, 2024

Also @jdyczka why are the actions failing? not just related to this PR

Copy link

❌ Docker Image Scan (Dockle) failed

No output

Copy link

✅ All checks succeeded

@Ryun1 Ryun1 marked this pull request as ready for review September 11, 2024 13:57
Copy link

✅ All checks succeeded

@Ryun1 Ryun1 force-pushed the 1976-fix-misleading-copy-for-direct-voter-registration branch from 3c4b74d to 8ec72a3 Compare September 12, 2024 11:40
@Ryun1 Ryun1 force-pushed the 1976-fix-misleading-copy-for-direct-voter-registration branch from 8ec72a3 to 0e3eeda Compare September 12, 2024 11:43
Copy link

✅ All checks succeeded

2 similar comments
Copy link

✅ All checks succeeded

Copy link

✅ All checks succeeded

@Ryun1 Ryun1 merged commit b520ab0 into develop Sep 12, 2024
8 checks passed
@Ryun1 Ryun1 deleted the 1976-fix-misleading-copy-for-direct-voter-registration branch September 12, 2024 11:49
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.

3 participants