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

2964 file type default #2965

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

wprice-uh
Copy link

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

Relevant Issues

resolves #2964

What is in this change?

Added an experimental setting labeled 'File Type Default' which, when turned on, changes the default behavior when a file being imported can't be recognized, from displaying an error message to treating the file as if it were plain text.

Additional Information

Happy to receive feedback and make changes as needed; I'm not well-versed in javascript or in the frameworks/libraries used here, so most of what I did was based on cloning the live-file-sync functionality.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

@c0mmand3rOpSec
Copy link

Thanks for this! Works great on my end, exactly what I was needing.

@timothycarambat
Copy link
Member

@wprice-uh

I'm not well-versed in javascript or in the frameworks/libraries used here

I dont believe that for a second, this looks great haha.

There is a general improvement that I will make on behalf of the PR, but this is great work - will be reviewing soon

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.

[FEAT]: Add setting to change default handling for unrecognized file types to '.txt'
3 participants