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

Add Personal View #391

Merged
merged 8 commits into from
Sep 29, 2023
Merged

Add Personal View #391

merged 8 commits into from
Sep 29, 2023

Conversation

jordycosta
Copy link
Member

@jordycosta jordycosta commented Sep 29, 2023

Overview:

Demo.mp4

This adds a button that will allow for filtering pulls that you have authored, wrote a signature on, or deploy/dev blocked

Additional functionality can easily be added with this


QA: Testing Documentation


Closes #383

This moves the UI component that displays the current connection status
along with the # of open pulls over to the right-hand side of the navigation
bar (before the Search bar).

I introduced a <Text> component from Chakra and leveraged a HStack to
align this section. I also separated the text of the "open:" and the
number of open pulls to make sure that there isn't a horizontal shift in
the user interface when the number of open pulls changes.
This function will return a boolean that indicates wether or not a pull
request is affiliated/associated with the Pulldasher user.

The current conditions to be considered associated:

- The pull request is authored by the user
- There is either a current or outdated CR signed by the user
- There is either a current or outdated QA signed by the user
- There is a dev_block signed by the user
- There is a deploy_block signed by the user
This setup is necessary for our new Personal View button to be able to
toggle the pull filtering on and off. The default state is false, i.e.
the personal view is not shown by default,.
This provides a check if a pull request is affiliated with the Pulldasher
user. This will be used with setPullFilter() in order to filter out pull
requests.
When the personal view is toggled on, this will filter the list of
pull requests to just those that are affiliated with the Pulldasher user.

All other pull requests will be hidden until the personal view is toggled
back off.
This will enable personal view filtering of pull request data.
This adds in hasMyDevBlock() and hasMyDeployBlock() to simplify reading
the isMineAffiliated() function. These functions will check if the Pulldasher
user has wrote a dev_block and/or deploy_block on a given pull request.
aburke07
aburke07 previously approved these changes Sep 29, 2023
Copy link
Contributor

@aburke07 aburke07 left a comment

Choose a reason for hiding this comment

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

CR 👽

@jordycosta
Copy link
Member Author

jordycosta commented Sep 29, 2023

@aburke07 I dropped the unused import in 2c8f4fe since I replaced that usage with Text in 8f78a51

Copy link
Contributor

@aburke07 aburke07 left a comment

Choose a reason for hiding this comment

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

CR 👽

@kthaler kthaler added the QAing Under QA team review label Sep 29, 2023
@kthaler
Copy link
Contributor

kthaler commented Sep 29, 2023

  • Selecting the "Show Personal View" and pulldasher showed everything I have authored, wrote a signature on, or deploy/dev blocked.
  • All the components work with the the Personal View option selected and unselected.
  • "Show Personal View" also filters the "Recently Closed Pulls"
  • Pulldasher continues to update in real time

QA 🦕 - This works great! Opens up the opportunity for so much new functionality.

@kthaler kthaler removed the QAing Under QA team review label Sep 29, 2023
@jordycosta jordycosta merged commit f40a597 into master Sep 29, 2023
1 check passed
@jordycosta jordycosta deleted the create-personal-view branch September 29, 2023 22:18
@danielbeardsley
Copy link
Member

Fantastic @jordycosta !!!

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.

Create Personal View
4 participants