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

Un-deprecate BsButton's buttonType argument #1110

Merged
merged 1 commit into from Jun 15, 2020
Merged

Conversation

simonihmig
Copy link
Contributor

Due to emberjs/ember.js#18232 we still support the attribute-argument buttonType in current 4.0 RCs, so remove the deprecation here to not cause folks to refactor to anything that will actually not work correctly.

Due to emberjs/ember.js#18232 we still support the attribute-argument in current 4.0 RCs, so remove the deprecation here to not cause folks to refactor to anything that will actually not work correctly.
@simonihmig simonihmig requested a review from jelhan June 12, 2020 10:01
@simonihmig simonihmig merged commit 8872523 into v3 Jun 15, 2020
@simonihmig simonihmig deleted the undeprecate-buttontype branch June 15, 2020 08:13
@jelhan
Copy link
Contributor

jelhan commented Jun 15, 2020

Very sad that this bug hasn't been fixed upstream yet. It's very likely to cause confusion. 😞

@RobbieTheWagner
Copy link

@simonihmig @jelhan so does this mean buttonType is required? We were using type and relying on splattributes, but that doesn't seem to work anymore.

@jelhan
Copy link
Contributor

jelhan commented Sep 15, 2020

Yes. The bug is still not fixed in Glimmer VM.

@RobbieTheWagner
Copy link

I believe buttonType is no longer documented. If that is the case, we should add it back to the docs.

@jelhan
Copy link
Contributor

jelhan commented Sep 16, 2020

I believe buttonType is no longer documented. If that is the case, we should add it back to the docs.

Good catch. Would appreciate a PR if you have the time. Documentation should be available in git history.

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

3 participants