Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[PLAY-1054] Use new sorting style #2846

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

markdoeswork
Copy link
Contributor

@markdoeswork markdoeswork commented Nov 6, 2023

What does this PR do? A clear and concise description with your runway ticket url.

Runway: https://nitro.powerhrg.com/runway/backlog_items/PLAY-1054

Removes the explicit Acending/Decending options from the header dropdown

Checkout the page this is used on in Nitro:
https://pr35577.nitro-web.beta.hq.powerapp.cloud/project_surveys/surveys?q%5Bcompleted_date_beginning_of_day_gteq%5D=01%2F01%2F2022&q%5Bcompleted_date_end_of_day_lteq%5D=12%2F31%2F2022&q%5Bs%5D=percentage_2+desc

I originally thought this would produce a breaking change to Nitro, but I opted not to change the shape of the sort_menu object. Therefore no breaking change 🎉

Screenshots: Screenshots to visualize your addition/change

From This:

screenshot-pr2846 playbook beta hq powerapp cloud-2023 11 07-09_38_41

To This:

screenshot-playbook powerapp cloud-2023 11 07-09_38_26

How to test? Steps to confirm the desired behavior:

  1. Go to https://pr2846.playbook.beta.hq.powerapp.cloud/kits/table/rails?sort=firstname_desc#table-header
  2. Click on the Full name header
  3. See addition/change

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.
  • TESTS I have added test coverage to my code.

@markdoeswork markdoeswork added breaking Indicates that a breaking change was introduced (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels Nov 6, 2023
@markdoeswork markdoeswork self-assigned this Nov 6, 2023
Copy link

github-actions bot commented Nov 6, 2023

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 13.10.0.pre.alpha.play1054tableheadersortconsolidation1416

Your Alpha for NPM is 13.10.0-alpha.play1054tableheadersortconsolidation1416

@markdoeswork markdoeswork added enhancement New Features, Props, & Variants (USED IN CHANGELOG) and removed breaking Indicates that a breaking change was introduced (USED IN CHANGELOG) labels Nov 7, 2023
@markdoeswork markdoeswork marked this pull request as ready for review November 7, 2023 14:40
@markdoeswork markdoeswork requested a review from a team as a code owner November 7, 2023 14:40
@nidaqg nidaqg added the Ready for Release merged to master, ready for a versioned released label Nov 21, 2023
@jasperfurniss jasperfurniss added this pull request to the merge queue Nov 21, 2023
Merged via the queue into master with commit a1a0075 Nov 21, 2023
@jasperfurniss jasperfurniss deleted the play/1054_table_header_sort_consolidation branch November 21, 2023 18:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
alpha enhancement New Features, Props, & Variants (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants