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

[transfer-engine] Add links support option. JB#55268 #3

Closed
wants to merge 1 commit into from

Conversation

mphilippov-omp
Copy link

No description provided.

@pvuorela
Copy link
Contributor

There's already a mime type for links (i.e. text/x-url), don't think we add redundant api like here.

@mphilippov-omp
Copy link
Author

There's already a mime type for links (i.e. text/x-url), don't think we add redundant api like here.

Well, I was asked to add it (in discussion)
https://os-git.omprussia.ru/jolla/base-transferengine-plugins/-/merge_requests/5

@pvuorela
Copy link
Contributor

So I'd rather go with a property for mimetypes that are not allowed, overriding any category/* wildcards.

@mphilippov-omp
Copy link
Author

mphilippov-omp commented Aug 25, 2021

So I'd rather go with a property for mimetypes that are not allowed, overriding any category/* wildcards.

So do you propose just text/*? Am I right?

@mphilippov-omp
Copy link
Author

@rainemak rainemak requested a review from blammit August 25, 2021 08:35
@pvuorela
Copy link
Contributor

Based on the discussion don't think we should be having this property

@pvuorela pvuorela closed this Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants