Skip to content

Conversation

@skjnldsv
Copy link
Member

Suggestion from @icewind1991: #52359 (comment)

This should allow us to show better files descriptions to users

  1. we are pulling those data from the freedesktop standard
  2. admins can add their own strings
  3. we can update this file regularly on demand

@skjnldsv skjnldsv added this to the Nextcloud 32 milestone May 14, 2025
@skjnldsv skjnldsv self-assigned this May 14, 2025
@skjnldsv skjnldsv requested review from a team as code owners May 14, 2025 16:15
@skjnldsv skjnldsv requested review from artonge, sorbaugh and susnux and removed request for a team May 14, 2025 16:15
@skjnldsv skjnldsv added 3. to review Waiting for reviews feature: files labels May 14, 2025
@skjnldsv skjnldsv requested review from ArtificialOwl and removed request for a team May 14, 2025 16:15
@skjnldsv skjnldsv force-pushed the feat/mime-names branch 2 times, most recently from 00731a7 to 5fe6407 Compare May 14, 2025 16:18
skjnldsv added 2 commits May 14, 2025 22:55
Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
Signed-off-by: John Molakvoæ (skjnldsv) <[email protected]>
@skjnldsv
Copy link
Member Author

@susnux ready for review

@skjnldsv skjnldsv requested a review from susnux May 15, 2025 07:04
@skjnldsv skjnldsv enabled auto-merge May 15, 2025 07:30
@skjnldsv skjnldsv merged commit cac44fd into master May 15, 2025
193 checks passed
@skjnldsv skjnldsv deleted the feat/mime-names branch May 15, 2025 09:41
themes: []
themes: [],
names: {'application/epub+zip': t('core', "Electronic book document"),
'application/gpx+xml': t('core', "GPX geographic data"),
Copy link
Member

Choose a reason for hiding this comment

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

do we want to group these translations in under core or have it under a separate "app"?

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.

6 participants