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

remove CoreBundle as we now ship the lat app that was referenced there #34190

Merged
merged 4 commits into from
Sep 22, 2022

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented Sep 22, 2022

No description provided.

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
- core bundle was empty and thus removed
- all former apps installed this way are shipped

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz added 3. to review Waiting for reviews technical debt labels Sep 22, 2022
@blizzz blizzz added this to the Nextcloud 26 milestone Sep 22, 2022
@blizzz blizzz requested review from nickvergessen, PVince81, skjnldsv, a team and CarlSchwan and removed request for a team September 22, 2022 11:33
@blizzz blizzz marked this pull request as ready for review September 22, 2022 11:33
@blizzz
Copy link
Member Author

blizzz commented Sep 22, 2022

/backport to stable25

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.

👍

@blizzz blizzz mentioned this pull request Sep 22, 2022
2 tasks
- because all apps are shipped now it was returning an empty result

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz blizzz force-pushed the fix/noid/remove-core-bundle branch from 47a9052 to 1b6ddb8 Compare September 22, 2022 13:11
@blizzz blizzz merged commit ac1c739 into master Sep 22, 2022
@blizzz blizzz deleted the fix/noid/remove-core-bundle branch September 22, 2022 15:34
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