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

[PBNTR-861] Advanced Table: Selectable Rows for Rails #4408

Merged
merged 6 commits into from
Mar 24, 2025

Conversation

ElisaShapiro
Copy link
Contributor

@ElisaShapiro ElisaShapiro commented Mar 19, 2025

What does this PR do? A clear and concise description with your runway ticket url.
PBNTR-861 is the first PR to get the Selectable Rows functionality to the Rails version of the Advanced Table via custom implementation.

This PR sets up new sets of json data as available for rails doc examples, enables adding a checkbox column to expandable and no subrow advanced tables with the appropriate styling (also contains a small dark mode fix present in prod for the React no subrow examples as well). They will be hidden from production after review until the follow up stories are completed. Full dynamic checkbox functionality for these two doc examples (with the expected cascading for expandable tables) and action bar functionality (added to these two doc ex and to the two additional examples React has) will come in follow up stories.

Screenshots: Screenshots to visualize your addition/change
selectable rows expandable
selectable rows expandable responsive
selectable rows no subrows
selectable rows no subrows responsive

How to test? Steps to confirm the desired behavior:

  1. Go to the rails advanced table selectable rows doc example in the review environment. Compare to React version of same doc ex, should look the same except for the checkbox cascade and row background change upon checkbox selection.
  2. Go to the rails advanced table selectable rows no subrows doc example in the review environment (link to come). Compare to React version of same doc ex, should look the same except for row background change upon checkbox selection.

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.

Sorry, something went wrong.

Verified

This commit was signed with the committer’s verified signature.
ElisaShapiro Elisa Shapiro

Verified

This commit was signed with the committer’s verified signature.
ElisaShapiro Elisa Shapiro

Verified

This commit was signed with the committer’s verified signature.
ElisaShapiro Elisa Shapiro
…in index.js
@ElisaShapiro ElisaShapiro added enhancement New Features, Props, & Variants (USED IN CHANGELOG) minor Semver Target labels Mar 19, 2025
@ElisaShapiro ElisaShapiro self-assigned this Mar 19, 2025

Verified

This commit was signed with the committer’s verified signature.
ElisaShapiro Elisa Shapiro
…ys into checkbox functionality
@powerhome-portal
Copy link

A change to documentation files was detected in your PR. Please visit this link to preview changes: https://portal-staging.powerapp.cloud/docs?filters[kind]=all&filters[user]=all&filters[namespaceFilter]=pbntr-861-adv-table-rails-selectable-rows

Verified

This commit was signed with the committer’s verified signature.
ElisaShapiro Elisa Shapiro
@ElisaShapiro ElisaShapiro added the milano 20 MAX - Deploy this PR to a review environment via Milano label Mar 20, 2025
@ElisaShapiro ElisaShapiro marked this pull request as ready for review March 20, 2025 17:36
@ElisaShapiro ElisaShapiro requested a review from a team as a code owner March 20, 2025 17:36
nidaqg
nidaqg previously approved these changes Mar 21, 2025
@nidaqg nidaqg added the Code Approved Approved by a Playbook Admin label Mar 21, 2025

Verified

This commit was signed with the committer’s verified signature.
ElisaShapiro Elisa Shapiro
@nidaqg nidaqg added Ready for Release merged to master, ready for a versioned released Product Approved pending technical review, OK to merge to master Inactive RC Skip the release candidate process labels Mar 24, 2025
@nidaqg nidaqg merged commit 62299b4 into master Mar 24, 2025
13 checks passed
@nidaqg nidaqg deleted the PBNTR-861-adv-table-rails-selectable-rows branch March 24, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Approved Approved by a Playbook Admin enhancement New Features, Props, & Variants (USED IN CHANGELOG) Inactive RC Skip the release candidate process milano 20 MAX - Deploy this PR to a review environment via Milano minor Semver Target Product Approved pending technical review, OK to merge to master Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants