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

Attest the ranked projects inside a category. #9

Open
7 tasks done
MoeNick opened this issue Oct 10, 2024 · 33 comments
Open
7 tasks done

Attest the ranked projects inside a category. #9

MoeNick opened this issue Oct 10, 2024 · 33 comments
Assignees

Comments

@MoeNick
Copy link
Contributor

MoeNick commented Oct 10, 2024

As a user, after I finish voting a Pairwise ranking and before I have been sent back to the categories page, I want to cast my vote on-chain by an attestation.

Design:

https://www.figma.com/design/tP7Arb2C8aDB05wCVxVVSB/Pairwise-RF6--%3E-Voting-App?node-id=1175-35870&t=vkHFl1L2Am6QdxHX-4

Success Criteria

NOTE: if the user has not voted on a category yet, logging in with an email modal is required.

@MoeNick MoeNick converted this from a draft issue Oct 10, 2024
@MoeNick MoeNick changed the title Attest the ranked projects inside all categories + allocation of categories Attest the ranked projects inside a category. Oct 15, 2024
@MoeNick
Copy link
Contributor Author

MoeNick commented Oct 15, 2024

@mmahdigh I updated the issue.
@Meriem-BM you can ask @mmahdigh to take care of the last step in pairwise comparisons.

@mmahdigh mmahdigh moved this from Todo to In Progress in Pairwise RF6 Oct 26, 2024
@MoeNick
Copy link
Contributor Author

MoeNick commented Oct 27, 2024

@mmahdigh I provided the link to the error attesting your vote as 7th success criteria.

@mmahdigh mmahdigh moved this from In Progress to On Staging / QA in Pairwise RF6 Oct 28, 2024
@maryjaf
Copy link
Collaborator

maryjaf commented Oct 28, 2024

By tapping on submit vote, I got an error @mmahdigh

Image

Payload:

Image

@MoeNick MoeNick moved this from On Staging / QA to In Progress in Pairwise RF6 Oct 28, 2024
@mmahdigh
Copy link
Collaborator

By tapping on submit vote, I got an error @mmahdigh

Image

Payload:

Image

Can you retest it now? The summation of your shares is 99.97% that's why our backend was declining it. I lowered the sensitivity for now.

@MoeNick MoeNick moved this from In Progress to On Staging / QA in Pairwise RF6 Oct 28, 2024
@mmahdigh
Copy link
Collaborator

By tapping on submit vote, I got an error @mmahdigh

Image

Payload:

Image

@Meriem-BM Could you take a look and investigate why is the summation 99.97%? My guess is that you're cutting share numbers to 4 decimal places (see the payload) and that's why we're short 0.03%. Let's keep at least 6 decimal places and only for the sake of showing the number in the input, show them rounded to 4 decimal places.

Let me know if it's something else or unavoidable.

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 28, 2024

By tapping on submit vote, I got an error @mmahdigh
Image
Payload:
Image

Can you retest it now? The summation of your shares is 99.97% that's why our backend was declining it. I lowered the sensitivity for now.

image


Screen.Recording.2024-10-28.at.4.38.09.PM.mov
Screen.Recording.2024-10-28.at.4.36.19.PM.mov

@maryjaf maryjaf moved this from On Staging / QA to In Progress in Pairwise RF6 Oct 28, 2024
@mmahdigh
Copy link
Collaborator

By tapping on submit vote, I got an error @mmahdigh
Image
Payload:
Image

Can you retest it now? The summation of your shares is 99.97% that's why our backend was declining it. I lowered the sensitivity for now.

image

Screen.Recording.2024-10-28.at.4.38.09.PM.mov
Screen.Recording.2024-10-28.at.4.36.19.PM.mov

Would you please give me your wallet address?

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 28, 2024

0xA1179f64638adb613DDAAc32D918EB6BEB824104 @mmahdigh

@mmahdigh
Copy link
Collaborator

0xA1179f64638adb613DDAAc32D918EB6BEB824104 @mmahdigh

Okay, It was a bug in calculating progress and finishing collections automatically. You gamed the system by giving every project a 1-star 😆

I reduced the number of comparisons for the time being so you can get to the end of comparisons faster.

I think you can successfully finish the attestation flow now.

@MoeNick
Copy link
Contributor Author

MoeNick commented Oct 28, 2024

@mmahdigh if you merge the @Meriem-BM PR then we test again

@mmahdigh
Copy link
Collaborator

She hasn't asked for a review yet. @MoeNick

@mmahdigh mmahdigh moved this from In Progress to On Staging / QA in Pairwise RF6 Oct 29, 2024
@maryjaf
Copy link
Collaborator

maryjaf commented Oct 29, 2024

Thanks @Meriem-BM @mmahdigh
This problem has been fixed

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

a new problem happens in attest flow @mmahdigh
when I submit vote, in allocation page I see the status of this category remains "vote" and it isn't changed to voted

https://github.com/user-attachments/assets/d6440d54-9293-4639-88a9-dc8691608b5a
https://github.com/user-attachments/assets/cdd6e0a5-ad78-4c9f-96f1-152383fa293e

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

a new problem happens in attest flow @mmahdigh when I submit vote, in allocation page I see the status of this category remains "vote" and it isn't changed to voted

https://github.com/user-attachments/assets/d6440d54-9293-4639-88a9-dc8691608b5a https://github.com/user-attachments/assets/cdd6e0a5-ad78-4c9f-96f1-152383fa293e

Thanks @Meriem-BM This problem has been fixed
But there is another issue , please take a look

  • If I tap on "back to category" before tapping on submit vote , in this page I see the status is change to voted and there is view attestation option
    @Meriem-BM

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

  • By tapping on "View attestation" link in allocation page, nothing happens @Meriem-BM
Screen.Recording.2024-10-30.at.2.23.59.PM.mov

@Meriem-BM
Copy link
Member

  • By tapping on "View attestation" link in allocation page, nothing happens @Meriem-BM

Screen.Recording.2024-10-30.at.2.23.59.PM.mov

@mmahdigh Do you know what should be triggered here? I haven't added anything yet, as well as for the Delegate button next to the budget bar.

@Meriem-BM
Copy link
Member

Meriem-BM commented Oct 30, 2024

  • If I tap on "back to category" before tapping on submit vote , in this page I see the status is change to voted and there is view attestation option

The View attestation should be shown when collection is attested? we have two status Finished (done voting) and Attested, if you can tell me what should we show for each? @MoeNick

@MoeNick
Copy link
Contributor Author

MoeNick commented Oct 30, 2024

  • If I tap on "back to category" before tapping on submit vote , in this page I see the status is change to voted and there is view attestation option

The View attestation should be shown when collection is attested? we have two status Finished (done voting) and Attested, if you can tell me what should we show for each? @MoeNick

Yes when attested we have attested status and we show the link.
Budget is a bit different as it does not have finished status
Finished is for category ranking, when the comparisons are done, but user did not click on attest yet. even in this case, the category is not voted and we cant have attestation link.
@Meriem-BM @mmahdigh

@Meriem-BM
Copy link
Member

Can you test these @maryjaf

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

  • If I tap on "back to category" before tapping on submit vote , in this page I see the status is change to voted and there is view attestation option

The View attestation should be shown when collection is attested? we have two status Finished (done voting) and Attested, if you can tell me what should we show for each? @MoeNick

Yes when attested we have attested status and we show the link. Budget is a bit different as it does not have finished status Finished is for category ranking, when the comparisons are done, but user did not click on attest yet. even in this case, the category is not voted and we cant have attestation link. @Meriem-BM @mmahdigh

this is ready for testing @Meriem-BM or only attest link ?

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

I haven't seen finished status for category, after completing ranking and before submitting vot I see below view
@Meriem-BM

image

@Meriem-BM
Copy link
Member

I haven't seen finished status for category, after completing ranking and before submitting vot I see below view @Meriem-BM

We didn't have that on the Figma designs, @MoeNick, should I add a status for case when comparison is done?

@MoeNick
Copy link
Contributor Author

MoeNick commented Oct 30, 2024

I haven't seen finished status for category, after completing ranking and before submitting vot I see below view @Meriem-BM

image

No we SHOULD NOT have finished status in UI, it's only voted. my comments was for backedn and finished / attested meaning.
We only have voted or vote in UI which means attested / not yet attested
@maryjaf @Meriem-BM

@MoeNick
Copy link
Contributor Author

MoeNick commented Oct 30, 2024

In above picture we should have Vote Button with Voting status. For Governance Leadership
means when user clicks, they can resume to edit / attest flow

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

In above picture we should have Vote Button with Voting status. means when user clicks, they can resume to edit / attest flow

@Meriem-BM

@Meriem-BM
Copy link
Member

@maryjaf you can test this one

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

I couldn't complete the submit vote flow because of this
#6 (comment)

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 30, 2024

In above picture we should have Vote Button with Voting status. For Governance Leadership means when user clicks, they can resume to edit / attest flow

It hasn't been fixed : @Meriem-BM

Screen.Recording.2024-10-31.at.12.07.22.AM.mov

@Meriem-BM
Copy link
Member

In above picture we should have Vote Button with Voting status. For Governance Leadership means when user clicks, they can resume to edit / attest flow

It hasn't been fixed : @Meriem-BM

Screen.Recording.2024-10-31.at.12.07.22.AM.mov

What's the status of that category @maryjaf? if it's done with comparison, then that's expected before it's attested. @MoeNick told me to redirect to last comparison and show that modal.

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 31, 2024

In above picture we should have Vote Button with Voting status. For Governance Leadership means when user clicks, they can resume to edit / attest flow

it's done with comparison(but not attested) and base on above comment the status should be voting and vote button should be shown

current behavior :

Screen.Recording.2024-10-31.at.8.52.04.PM.mov

@Meriem-BM @MoeNick

@Meriem-BM
Copy link
Member

In above picture we should have Vote Button with Voting status. For Governance Leadership means when user clicks, they can resume to edit / attest flow

it's done with comparison(but not attested) and base on above comment the status should be voting and vote button should be shown

current behavior :

Screen.Recording.2024-10-31.at.8.52.04.PM.mov
@Meriem-BM @MoeNick

Got it, my bad, I should keep the voting status

@Meriem-BM
Copy link
Member

@maryjaf can you check this please

@Meriem-BM Meriem-BM moved this from In Progress to On Staging / QA in Pairwise RF6 Oct 31, 2024
@maryjaf
Copy link
Collaborator

maryjaf commented Oct 31, 2024

Thanks @Meriem-BM

current behavior :
cc: @MoeNick

image

@maryjaf maryjaf moved this from On Staging / QA to Approved by QA in Pairwise RF6 Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Merged on Production
Development

No branches or pull requests

4 participants