Skip to content

Conversation

@julien-nc
Copy link
Member

AppSidebar primary-actions slot became description in @nc/vue v3.3.0 (nextcloud-libraries/nextcloud-vue#1566).
This PR adapts the slot name in templates when this component is used in app settings and in files.

I've noticed it's also used in Calendar app but @nc/vue is still v2.9.0 there.

Let's keep that in mind in case @nc/vue version is bumped in stable21 😁.

Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

🙈
Thanks 🙏

@julien-nc
Copy link
Member Author

/compile amend /

@npmbuildbot-nextcloud npmbuildbot-nextcloud bot force-pushed the fix/app-sidebar-description-slot branch from e3a2e6c to 9d1d95e Compare March 18, 2021 11:23
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Mar 18, 2021
@PVince81
Copy link
Member

/compile amend /

@rullzer rullzer force-pushed the fix/app-sidebar-description-slot branch from 9d1d95e to 3c172ce Compare March 19, 2021 11:56
@rullzer
Copy link
Member

rullzer commented Mar 19, 2021

Removed the extra settings files
We can merge once done

…me 'description' in @nc/vue v3.3.0

Signed-off-by: Julien Veyssier <[email protected]>
Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
@julien-nc julien-nc merged commit 086cf56 into master Mar 19, 2021
@julien-nc julien-nc deleted the fix/app-sidebar-description-slot branch March 19, 2021 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug feature: files feature: settings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants