Skip to content

[File upload] Enforce file-type check in file dialog - #55063

Merged
thomasneirynck merged 4 commits into
elastic:masterfrom
thomasneirynck:fileupload/restrict_to_json
Jan 21, 2020
Merged

thomasneirynck merged 4 commits into
elastic:masterfrom
thomasneirynck:fileupload/restrict_to_json

Conversation

@thomasneirynck

Copy link
Copy Markdown
Contributor

Closes #53912

Upload is already rejecting the file if not .json/.geojson. We shouldn't be showing these files the dialog.

@thomasneirynck thomasneirynck added bug Fixes for quality problems that affect the customer experience Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.6.0 v7.7.0 v8.0.0 labels Jan 16, 2020
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-gis (Team:Geo)

@kindsun kindsun changed the title [Fileupload] Enforce file-type check in file dialog [File upload] Enforce file-type check in file dialog Jan 16, 2020

@nreese nreese left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm
code review, tested in chrome

@kindsun kindsun left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

  • code review
  • tested locally in chrome

@thomasneirynck

Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@thomasneirynck

Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience Feature:File Upload release_note:fix Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v7.6.0 v7.7.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Maps] Only show files with extensions that are supported when uploading geojson

5 participants