Skip to content

[Maps][File upload] Remove usage of max file size advanced setting 1GB limit in geo file upload#127639

Merged
nreese merged 9 commits intoelastic:mainfrom
maksimkovalev:e-93261
Mar 16, 2022
Merged

[Maps][File upload] Remove usage of max file size advanced setting 1GB limit in geo file upload#127639
nreese merged 9 commits intoelastic:mainfrom
maksimkovalev:e-93261

Conversation

@maksimkovalev
Copy link
Copy Markdown
Contributor

@maksimkovalev maksimkovalev commented Mar 14, 2022

Closes: #93261

The file size check has been removed for geo file upload form. The message about file size limit has also been removed from geo upload wizard.

@maksimkovalev maksimkovalev added Team:Geo Former Team Label for Geo Team. Now use Team:Presentation enhancement New value added to drive a business result Feature:File Upload backport:skip This PR does not require backporting v8.2.0 labels Mar 14, 2022
@maksimkovalev maksimkovalev requested a review from nreese March 14, 2022 18:26
Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

Can you update the docs to remove from "TIP: When you upload files in {kib}, there is a file size limit, which is configurable in <<fileupload-maxfilesize,Advanced Settings>>." https://github.com/elastic/kibana/blob/main/docs/maps/import-geospatial-data.asciidoc

@maksimkovalev maksimkovalev requested a review from nreese March 15, 2022 19:48
@nreese nreese marked this pull request as ready for review March 16, 2022 19:44
@nreese nreese requested a review from a team as a code owner March 16, 2022 19:44
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@nreese
Copy link
Copy Markdown
Contributor

nreese commented Mar 16, 2022

@elasticmachine merge upstream

Copy link
Copy Markdown
Contributor

@nreese nreese left a comment

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

@nreese nreese added release_note:enhancement and removed enhancement New value added to drive a business result labels Mar 16, 2022
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
fileUpload 821.2KB 821.1KB -105.0B

History

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

@nreese nreese merged commit 0cd251e into elastic:main Mar 16, 2022
maksimkovalev added a commit to maksimkovalev/kibana that referenced this pull request Mar 18, 2022
…B limit in geo file upload (elastic#127639)

* elastic#93261 - Usage of maxFileSize in geo upload form is removed

* 93261 - refactoring

* 93261 - fixed translation checks

* 93261 - documentation updates

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting Feature:File Upload release_note:enhancement Team:Geo Former Team Label for Geo Team. Now use Team:Presentation v8.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Maps][File upload] Remove usage of max file size advanced setting 1GB limit in geo file upload

5 participants