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

[stable21] Fix comments file action sidebar opening #28198

Merged
merged 1 commit into from
Jul 28, 2021

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Jul 27, 2021

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #27893

@julien-nc
Copy link
Member

/compile amend /

@nextcloud-bot nextcloud-bot force-pushed the backport/27893/stable21 branch from acdd7e7 to 21ca868 Compare July 27, 2021 15:45
@julien-nc julien-nc force-pushed the backport/27893/stable21 branch from 21ca868 to 482c680 Compare July 27, 2021 16:05
@julien-nc
Copy link
Member

/compile amend /

@nextcloud-bot nextcloud-bot force-pushed the backport/27893/stable21 branch from 482c680 to 56c5c41 Compare July 27, 2021 16:18
@julien-nc julien-nc force-pushed the backport/27893/stable21 branch from 56c5c41 to fc8ab56 Compare July 27, 2021 21:34
@julien-nc julien-nc force-pushed the backport/27893/stable21 branch from fc8ab56 to 6bb95ee Compare July 28, 2021 08:15
@julien-nc
Copy link
Member

The checks for stable22 went well but this one is resisting 😕

The static code analysis issues are unrelated.

If compilation is done by the compile bot, the node build check complains about "Uncommited changes in webpack build" which concerns the comments-app.js and comments-app.js.map which are not impacted by the changes of this PR.
So in the end this check passes after having pushed a commit in which those 2 files are not recompiled.

I think this can be merged.

@julien-nc julien-nc merged commit fb224dd into stable21 Jul 28, 2021
@julien-nc julien-nc deleted the backport/27893/stable21 branch July 28, 2021 09:44
@julien-nc
Copy link
Member

Thanks @artonge

@skjnldsv skjnldsv mentioned this pull request Aug 3, 2021
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants