Skip to content

Conversation

@skjnldsv
Copy link
Member

Backport of #421

@skjnldsv skjnldsv added bug Something isn't working 3. to review Waiting for reviews labels Mar 19, 2020
@skjnldsv skjnldsv requested a review from danxuliu March 19, 2020 09:34
@skjnldsv skjnldsv self-assigned this Mar 19, 2020
@skjnldsv skjnldsv changed the base branch from master to stable18 March 19, 2020 09:34
@skjnldsv skjnldsv added this to the Nextcloud 18.0.3 milestone Mar 19, 2020
@skjnldsv
Copy link
Member Author

/compile amend /

When OCA.Files is not defined the sidebar is not available, but in that
case it is not possible to check against OCA.Files.Sidebar.

When the sidebar is not available "sidebarFile" is null, and when the
sidebar is available but there is no file it is an empty string, so it
is enough to check if "sidebarFile" has a value or not in order to open
the sidebar or not.

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
@npmbuildbot-nextcloud npmbuildbot-nextcloud bot force-pushed the backport/421/fix-trying-to-open-the-sidebar-when-not-available branch from 17c1932 to 4ef4d6f Compare March 19, 2020 09:36
@cypress
Copy link

cypress bot commented Mar 19, 2020



Test summary

135 0 0 0


Run details

Project viewer
Status Passed
Commit f403ea2 ℹ️
Started Mar 19, 2020 9:39 AM
Ended Mar 19, 2020 9:41 AM
Duration 01:41 💡
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 skjnldsv merged commit 2f97d31 into stable18 Mar 19, 2020
@skjnldsv skjnldsv deleted the backport/421/fix-trying-to-open-the-sidebar-when-not-available branch March 19, 2020 10:31
@rullzer rullzer mentioned this pull request Apr 13, 2020
7 tasks
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants