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

Public pages compatibility #408

Merged
merged 1 commit into from
Mar 12, 2020
Merged

Public pages compatibility #408

merged 1 commit into from
Mar 12, 2020

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Feb 28, 2020

Fix #89

Use it with nextcloud/server#19777

@skjnldsv skjnldsv self-assigned this Feb 28, 2020
@skjnldsv skjnldsv added 2. developing Work in progress 3. to review Waiting for reviews enhancement New feature or request high High priority labels Feb 28, 2020
@cypress

This comment has been minimized.

@skjnldsv skjnldsv removed the 2. developing Work in progress label Mar 4, 2020
@skjnldsv skjnldsv marked this pull request as ready for review March 4, 2020 12:53
@skjnldsv
Copy link
Member Author

skjnldsv commented Mar 4, 2020

Works fine!

Please test with nextcloud/server#19777

I forgot to stop listening to LoadAdditionalScriptsEvent like we do with the LoadSidebar event. This is enough :)

src/mixins/Mime.js Outdated Show resolved Hide resolved
Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

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

Tested and works

lib/AppInfo/Application.php Show resolved Hide resolved
@skjnldsv
Copy link
Member Author

skjnldsv commented Mar 4, 2020

/compile amend /

@skjnldsv skjnldsv 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 4, 2020
@cypress
Copy link

cypress bot commented Mar 4, 2020



Test summary

135 0 0 0


Run details

Project viewer
Status Passed
Commit a0b40d0
Started Mar 12, 2020 8:31 AM
Ended Mar 12, 2020 8:33 AM
Duration 01:35 💡
OS Linux Ubuntu Linux - 18.04
Browser Electron 78

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@skjnldsv
Copy link
Member Author

/backport to stable18

@backportbot-nextcloud backportbot-nextcloud bot added the backport-request Pending backport by the backport-bot label Mar 12, 2020
@skjnldsv
Copy link
Member Author

/compile amend /

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
@skjnldsv skjnldsv merged commit ded5b1a into master Mar 12, 2020
@skjnldsv skjnldsv deleted the enh/public branch March 12, 2020 09:19
@backportbot-nextcloud
Copy link

The backport to stable18 failed. Please do this backport manually.

@skjnldsv skjnldsv removed the backport-request Pending backport by the backport-bot label Mar 12, 2020
@skjnldsv
Copy link
Member Author

backport in #422

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 enhancement New feature or request high High priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Viewer also for public links
3 participants