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

gpasc-radio-button-toggle.js: Added snippet to toggle GPASC auto save client side using radio buttons. #631

Merged
merged 3 commits into from
Sep 21, 2023

Conversation

veryspry
Copy link
Contributor

@veryspry veryspry commented Feb 20, 2023

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2151546100/44070?folderId=6987275

Summary

Adds snippet to allow radio buttons to toggle ASC on/off.

See also the required sister code change within GPASC: https://github.com/gravitywiz/gp-advanced-save-and-continue/pull/9

@veryspry veryspry force-pushed the matt/gpasc-radio-button-toggle branch 2 times, most recently from 17f32d4 to e736e36 Compare February 21, 2023 18:41
@veryspry veryspry force-pushed the matt/gpasc-radio-button-toggle branch from e736e36 to aa9d059 Compare February 22, 2023 22:58
@veryspry veryspry changed the title gpasc-radio-button-toggle.js: Added snippet to toggle GPASC auto save client side using radio buttons. gpasc-radio-button-toggle.js: Added snippet to toggle GPASC auto save client side using radio buttons. Feb 22, 2023
Copy link
Contributor

@claygriffiths claygriffiths left a comment

Choose a reason for hiding this comment

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

Good stuff!

@veryspry
Copy link
Contributor Author

veryspry commented Sep 21, 2023

@claygriffiths I made those suggested changes (and tested them ofc) so I'm going to go ahead and merge since (to my knowledge) haven't seen any issues with this particular snippet from the user I sent this to.

@veryspry veryspry merged commit 279811b into master Sep 21, 2023
3 checks passed
@veryspry veryspry deleted the matt/gpasc-radio-button-toggle branch September 21, 2023 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants