Skip to content

Conversation

@juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Aug 4, 2020

For nextcloud/server#20930

Once merged:

  • Update resources/app-info.xsd in server as well nextcloud/server@b88b86e
  • Add app store category to supporting apps info.xml

{% trans "Customization" %}
{% trans "Themes, layout and UX change apps" %}
{% trans "Dashboard" %}
{% trans "Apps for Nextcloud Dashboard" %}
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

"fields": {
"language_code": "en",
"name": "Dashboard",
"description": "Apps for Nextcloud Dashboard",
Copy link
Member

Choose a reason for hiding this comment

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

I think this category is for apps that "include" a dashboard widget but that are potentially not specialized in that area. What about changing this to
Apps including Nextcloud Dashboard widgets ?

@coveralls
Copy link

coveralls commented Aug 4, 2020

Coverage Status

Coverage remained the same at 94.386% when pulling 6e675c0 on enh/noid/dashboard into c4d3206 on master.

Signed-off-by: Julius Härtl <[email protected]>
@juliusknorr juliusknorr dismissed julien-nc’s stale review August 4, 2020 14:24

Good point. Fixed.

@MorrisJobke MorrisJobke merged commit 1d9c9cd into master Aug 5, 2020
@delete-merged-branch delete-merged-branch bot deleted the enh/noid/dashboard branch August 5, 2020 06:39
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.

5 participants