Skip to content

feat: Add text/plain MIME type mapping for 'text' extension #54737

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.

Sign up for GitHub

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

Jump to bottom
Merged
marcelklehr merged 1 commit into from
Sep 1, 2025
Merged

feat: Add text/plain MIME type mapping for 'text' extension #54737

marcelklehr merged 1 commit into from
Sep 1, 2025

Conversation

@marcelklehr
Copy link
Member

@marcelklehr marcelklehr commented Aug 29, 2025

Checklist

@marcelklehr marcelklehr requested a review from a team as a code owner August 29, 2025 11:53
@marcelklehr marcelklehr requested review from ArtificialOwl, come-nc and leftybournes and removed request for a team August 29, 2025 11:53
@AndyScherzinger AndyScherzinger added this to the Nextcloud 32 milestone Aug 30, 2025
@AndyScherzinger
Copy link
Member

AndyScherzinger commented Aug 30, 2025

I guess it needs further changes, like dea865d

AndyScherzinger
AndyScherzinger reviewed Sep 1, 2025
View reviewed changes
@susnux
Copy link
Contributor

susnux commented Sep 1, 2025

Where is a .text extension used? As I never saw this before, only .txt for plain text files.

@marcelklehr
Copy link
Member Author

marcelklehr commented Sep 1, 2025

It was requested by a customer and is documented in Wikipedia as an alternative to txt, so I thought I might add it 🤷

provokateurin
provokateurin requested changes Sep 1, 2025
View reviewed changes
@marcelklehr marcelklehr merged commit 61bb2dc into master Sep 1, 2025
198 checks passed
@marcelklehr marcelklehr deleted the feat/text-ext branch September 1, 2025 11:36
@susnux
Copy link
Contributor

susnux commented Sep 1, 2025

It was requested by a customer and is documented in Wikipedia as an alternative to txt, so I thought I might add it 🤷

All good - I was just curious :)

@nextcloud-bot nextcloud-bot mentioned this pull request Sep 2, 2025
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@provokateurin provokateurin provokateurin approved these changes

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl is a code owner automatically assigned from nextcloud/server-backend

@leftybournes leftybournes Awaiting requested review from leftybournes leftybournes is a code owner automatically assigned from nextcloud/server-backend

@come-nc come-nc Awaiting requested review from come-nc come-nc is a code owner automatically assigned from nextcloud/server-backend

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

6 participants

@marcelklehr @AndyScherzinger @susnux @provokateurin @skjnldsv