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

Use a Stimulus controller value for success content #10

Merged
merged 4 commits into from
Jun 1, 2023

Conversation

jmsche
Copy link
Contributor

@jmsche jmsche commented May 16, 2023

Hi,

I was a little surprised to see that the "success content" wasn't a Stimulus controller value, so here it is :)

Would require a new major release.

Didn't find the docs though, I guess they are located in a Stimulus Components website repository?

@guillaumebriday
Copy link
Member

Can you fix the conflicts so I can merge? Thanks 🙏

@guillaumebriday
Copy link
Member

And I will release a new version with your previous PR and this one 👍

@jmsche
Copy link
Contributor Author

jmsche commented May 22, 2023

Hi, just fixed the conflicts :)

@guillaumebriday guillaumebriday merged commit 4f2f796 into stimulus-components:master Jun 1, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants