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

Fixes motions not appearing in the correct domain in actions list #3073

Conversation

arrenv
Copy link
Member

@arrenv arrenv commented Dec 23, 2021

Description

Fixes motions correctly appearing in the actions list and actions team filter. See this issue for details #3056.

New stuff

  • Adds assignment of ethDomainId to motion action item.
  • Adds expectation check to actions motion transformer.

Testing

  • Ensure the 'Motions' extension is enabled.
  • Create a child team.
  • You will need to make sure there is funds and reputation in the team.
  • Perform a motion action in the child team and ensure it finishes. You can speed up the time with the following script, this passes 3 days of time. Just run it in your terminal.
    curl -H "Content-Type: application/json" -X POST --data '{"jsonrpc":"2.0","method":"evm_increaseTime","params":[260000],"id": 1}' localhost:8545 && curl -H "Content-Type: application/json" -X POST --data '{"jsonrpc":"2.0","method":"evm_mine","params":[]}' localhost:8545
  • Ensure the motion action appears in the list. It may take a second to load after the other actions load.
  • Check the domain label of the action.
  • Filter the actions by the team, and ensure the motion appears in the list.

Before

actions-wrong-team-2

actions-wrong-team

After ✨

actions-right-team

actions-right-team-2

Resolves #3056

@arrenv arrenv added the bug label Dec 23, 2021
@arrenv arrenv self-assigned this Dec 23, 2021
@arrenv arrenv requested a review from a team December 23, 2021 17:23
Copy link
Contributor

@chinins chinins left a comment

Choose a reason for hiding this comment

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

It doesn't seem like it's working for me....

This is the motion I'm creating:
Screenshot 2021-12-23 at 16 02 16

However the motion created in is Root and not the cave of wonders
Screenshot 2021-12-23 at 16 02 52

And here is this motion in the list (and actually the one before was also in the cave of wonders team, I just created another one to double check):
Screenshot 2021-12-23 at 16 03 46

And when i'm filtering by domain neither of the motions are there:
Screenshot 2021-12-23 at 16 05 22

And here is my terminal confirmation that I'm on the correct branch:
Screenshot 2021-12-23 at 16 06 09

@arrenv
Copy link
Member Author

arrenv commented Dec 24, 2021

@chinins I thought that may have been a bug, that is why I had trouble replicating the issue initially.

This is a separate issue on allocating the correct team from the dialog, however, I can look into it in this issue as well. What do you think?

This PR is focused on getting the right information from the SubGraph. To create a motion in a domain, you need to select the team before creating the action. See gif below:

correct-team-allocation

@chinins
Copy link
Contributor

chinins commented Dec 24, 2021

@chinins I thought that may have been a bug, that is why I had trouble replicating the issue initially.

This is a separate issue on allocating the correct team from the dialog, however, I can look into it in this issue as well. What do you think?

This PR is focused on getting the right information from the SubGraph. To create a motion in a domain, you need to select the team before creating the action. See gif below:

correct-team-allocation

We had a call with Arren and clarified where my problem was. It seems that there is a bug in CreatePayment dialog that doesn't send correct motion domain to the network. It is indeed a separate issue which I will create.

Copy link
Contributor

@chinins chinins left a comment

Choose a reason for hiding this comment

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

It's all clear and good now.

@arrenv arrenv force-pushed the fix/3056-motions-not-appearing-in-the-correct-domain-in-actions branch from d527b7b to f3a03a7 Compare January 4, 2022 10:23
@arrenv arrenv merged commit f09f84c into master Jan 4, 2022
@arrenv arrenv deleted the fix/3056-motions-not-appearing-in-the-correct-domain-in-actions branch January 4, 2022 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Passed motions from within a team, don't appear in the team filtered view
3 participants