Skip to content

Conversation

@jancborchardt
Copy link
Member

Before

recommended apps before

After

recommended apps after

Fix #18363, please review @ChristophWurst @rullzer @juliushaertl

@jancborchardt jancborchardt added bug design Design, UI, UX, etc. 3. to review Waiting for reviews labels Dec 16, 2019
@jancborchardt jancborchardt added this to the Nextcloud 18 milestone Dec 16, 2019
Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

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

Screenshots looks good

Bundle was not updated

@ChristophWurst
Copy link
Member

/compile amend /

@rullzer rullzer force-pushed the design/recommended-apps branch from f70755e to 2ebfdcf Compare December 16, 2019 14:06
@rullzer
Copy link
Member

rullzer commented Dec 16, 2019

/compile amend /

Signed-off-by: Jan-Christoph Borchardt <[email protected]>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
@jancborchardt
Copy link
Member Author

@rullzer from what I can tell, the Drone failures are unrelated?

@jancborchardt
Copy link
Member Author

Also thanks to @gary-kim for the in-person review! :) Feel free to check the code if you like

@rullzer rullzer mentioned this pull request Dec 17, 2019
34 tasks
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Dec 17, 2019
@rullzer rullzer merged commit 53f94b8 into master Dec 17, 2019
@rullzer rullzer deleted the design/recommended-apps branch December 17, 2019 11:02
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 design Design, UI, UX, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dedicated app install page polishing

4 participants