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

feat(check-question): add reset button for optional question #2549

Merged
merged 8 commits into from
Dec 19, 2023

Conversation

amchangama
Copy link
Contributor

@amchangama amchangama commented Nov 20, 2023

Description

Adds a reset button after non-required inputs of a choice question type.

Preview

image

Copy link
Contributor

@derrabauke derrabauke left a comment

Choose a reason for hiding this comment

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

I've added a little PR description+preview. Code looks good 👍

@czosel czosel changed the title feat(check-question): add rest button for optional question feat(check-question): add reset button for optional question Dec 8, 2023
@derrabauke derrabauke force-pushed the reset-selected-option branch 2 times, most recently from 97fb5cc to 3ba0248 Compare December 18, 2023 21:44
packages/form/addon/components/cf-field/input/radio.hbs Outdated Show resolved Hide resolved
packages/form/addon/components/cf-field/input/radio.hbs Outdated Show resolved Hide resolved
packages/form/addon/components/cf-field/input/radio.hbs Outdated Show resolved Hide resolved
packages/form/package.json Show resolved Hide resolved
@anehx
Copy link
Contributor

anehx commented Dec 19, 2023

@derrabauke You forgot to push.. There are no changes after my review.

@derrabauke
Copy link
Contributor

@derrabauke You forgot to push.. There are no changes after my review.

Not forgotten, but I was tangled in confusion how the ember-truth-helpers references get resolved in the app and if it should be a dependency or not. 🤯

@derrabauke derrabauke force-pushed the reset-selected-option branch from f87a06f to a168022 Compare December 19, 2023 12:26
@anehx
Copy link
Contributor

anehx commented Dec 19, 2023

Alright, one final nitpick then it is ready to go 🎉 Sorry @derrabauke 😉

@anehx anehx merged commit f9c490f into projectcaluma:main Dec 19, 2023
57 checks passed
@anehx
Copy link
Contributor

anehx commented Dec 19, 2023

🎉 This PR is included in version 12.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants