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

small fixes for the Nextcloud 10 release #2

Merged
merged 1 commit into from
Aug 10, 2016
Merged

Conversation

schiessle
Copy link
Member

see nextcloud/server#735

needs backport to stable10 once done

@schiessle schiessle added this to the Nextcloud 11 milestone Aug 8, 2016
@schiessle
Copy link
Member Author

schiessle commented Aug 8, 2016

@jancborchardt @MorrisJobke please review....

There are a few items left:

  • confirm button to right: I didn't find a way to change the order of the buttons. But the admin only sees this notification once in is life. So I would say that this point is not super urgent
  • add link to survey notification: Don't remember what was meant here. You can click the notification to go to the corresponding admin settings. What other link do we want?
  • line-height needs to be fixed: no idea what was meant here, looks good to me

This is how it looks now:

usage report notification

usage report

@schiessle schiessle modified the milestones: Nextcloud 11.0, Nextcloud 11 Aug 8, 2016
@schiessle schiessle merged commit a0a2a16 into master Aug 10, 2016
@schiessle schiessle deleted the get-ready-for-release branch August 10, 2016 09:44
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.

1 participant