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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃悰 (#2529): fix missing firstAction #2532

Merged
merged 1 commit into from
Mar 3, 2022
Merged

Conversation

vinicius73
Copy link
Contributor

fixes #2529

fix reactivity of children slots
Copy link
Contributor

@juliushaertl juliushaertl left a comment

Choose a reason for hiding this comment

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

Great, much nicer than the template wrapping workaround from yesterday 馃憤

Can also be tested by comparing https://nextcloud-vue-components.netlify.app/#/Components/Modal with https://deploy-preview-2532--nextcloud-vue-components.netlify.app/#/Components/Modal

@juliushaertl
Copy link
Contributor

@vinicius73 Could you check https://github.com/nextcloud/nextcloud-vue/pull/2532/checks?check_run_id=5398774806 on how address the missing DCO Signed-off-by message?

@skjnldsv skjnldsv merged commit 8fdc0d2 into master Mar 3, 2022
@skjnldsv skjnldsv deleted the feature/fix-2529 branch March 3, 2022 13:10
@skjnldsv skjnldsv added 3. to review Waiting for reviews bug Something isn't working design Design, UX, interface and interaction design feature: actions Related to the actions components regression Regression of a previous working feature labels Mar 3, 2022
@szaimen szaimen added this to the 5.1.1 milestone Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug Something isn't working design Design, UX, interface and interaction design feature: actions Related to the actions components regression Regression of a previous working feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Modal close button missing after 5.x
5 participants