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-456] Fixed Confirmation Toast - Enable "auto close" for RAILS #4179

Merged
merged 8 commits into from
Jan 31, 2025

Conversation

ElisaShapiro
Copy link
Contributor

@ElisaShapiro ElisaShapiro commented Jan 27, 2025

What does this PR do? A clear and concise description with your runway ticket url.
PBNTR-456 requested creation of an auto_close prop for the Rails Fixed Confirmation Toast; investigation revealed that PBNTR-475 actually created both React and Rails versions of the prop, but only displayed a doc example for the React side. It is already being used in Nitro in a few places see here.

This PR adds the rails doc example and updates the PbEnhancedElement file to ensure autoclosing works as expected for standard and closeable (has X to close) toasts. An initial overhaul of the PbEnhancedElement was too much and failed alpha testing (which revealed that the previously only closeable autoclose fixed confirmation toasts worked in nitro at all) - this PR contains a minimally updated version that works for both closeable and default autoclose toasts in rails.

Screenshots: Screenshots to visualize your addition/change
for PR 3 seconds both work in alpha
for PR 10 seconds both work in alpha

How to test? Steps to confirm the desired behavior:

  1. Go to the rails autoclose doc example.
  2. Click on buttons to trigger autocloseable toasts.

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.

@ElisaShapiro ElisaShapiro added milano 20 MAX - Deploy this PR to a review environment via Milano improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) minor Semver Target labels Jan 27, 2025
@ElisaShapiro ElisaShapiro self-assigned this Jan 27, 2025
@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-456-fixed-conf-toast-rails-autoclose

@ElisaShapiro ElisaShapiro marked this pull request as ready for review January 27, 2025 15:45
@ElisaShapiro ElisaShapiro requested a review from a team as a code owner January 27, 2025 15:45
@nidaqg
Copy link
Contributor

nidaqg commented Jan 27, 2025

Hey @ElisaShapiro , code looks good, but when you first navigate to the Rails side of the fixed confirmation toast, the toast for 'I will close in 10s' is already on the page for a second before it disappears. Not sure if this is kit level issue or a doc script tag issue

nidaqg
nidaqg previously approved these changes Jan 28, 2025
@nidaqg nidaqg added the Code Approved Approved by a Playbook Admin label Jan 28, 2025
Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.12.0.pre.alpha.PBNTR456fixedconftoastrailsautoclose5673

Your Alpha for NPM is 14.12.0-alpha.PBNTR456fixedconftoastrailsautoclose5673

Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.12.0.pre.alpha.PBNTR456fixedconftoastrailsautoclose5728

Your Alpha for NPM is 14.12.0-alpha.PBNTR456fixedconftoastrailsautoclose5728

Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.12.0.pre.alpha.PBNTR456fixedconftoastrailsautoclose5738

Your Alpha for NPM is 14.12.0-alpha.PBNTR456fixedconftoastrailsautoclose5738

Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.12.0.pre.alpha.PBNTR456fixedconftoastrailsautoclose5754

Your Alpha for NPM is 14.12.0-alpha.PBNTR456fixedconftoastrailsautoclose5754

Copy link

🎉 Congratulations on creating an Alpha Version!

Your Alpha for Ruby Gems is 14.12.0.pre.alpha.PBNTR456fixedconftoastrailsautoclose5757

Your Alpha for NPM is 14.12.0-alpha.PBNTR456fixedconftoastrailsautoclose5757

@ElisaShapiro ElisaShapiro merged commit 7dae173 into master Jan 31, 2025
7 checks passed
@ElisaShapiro ElisaShapiro deleted the PBNTR-456-fixed-conf-toast-rails-autoclose branch January 31, 2025 13:18
Copy link

You merged this pr to master branch:
- Ruby Gem: 14.13.0.pre.rc.6
- NPM: 14.13.0-rc.6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alpha Code Approved Approved by a Playbook Admin improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) milano 20 MAX - Deploy this PR to a review environment via Milano minor Semver Target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants