Skip to content

Prevent space identifier url example from overflowing panel - #44871

Merged
legrego merged 4 commits into
elastic:masterfrom
legrego:spaces/max-id-length
Sep 5, 2019
Merged

legrego merged 4 commits into
elastic:masterfrom
legrego:spaces/max-id-length

Conversation

@legrego

@legrego legrego commented Sep 5, 2019

Copy link
Copy Markdown
Member

Summary

Fixes #44849

image

@legrego legrego added release_note:fix Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t// Feature:Security/Spaces Platform Security - Spaces feature v8.0.0 v7.4.0 v7.5.0 labels Sep 5, 2019
@legrego
legrego requested a review from a team as a code owner September 5, 2019 11:59
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-security

@legrego
legrego requested review from azasypkin and kobelb September 5, 2019 12:00
@azasypkin

Copy link
Copy Markdown
Contributor

ACK: reviewing now. Btw, looks like we need to update Jest snapshot @legrego

@azasypkin azasypkin 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, tested locally and everything looks good, thanks! As we discussed over slack we may need to file EUI issue related to the way EUI inputs ^handles^ with very long non-ASCII strings.

@legrego
legrego removed the request for review from kobelb September 5, 2019 13:10
@elasticmachine

This comment has been minimized.

@elasticmachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@legrego
legrego merged commit 1a15d5b into elastic:master Sep 5, 2019
@legrego
legrego deleted the spaces/max-id-length branch September 5, 2019 14:06
legrego added a commit to legrego/kibana that referenced this pull request Sep 5, 2019
…44871)

* ensure example url wraps properly when the space identifier is long

* restrict space identifier length to 500 characters

* Revert "restrict space identifier length to 500 characters"

This reverts commit a34f324.

* update snapshot
legrego added a commit that referenced this pull request Sep 5, 2019
* ensure example url wraps properly when the space identifier is long

* restrict space identifier length to 500 characters

* Revert "restrict space identifier length to 500 characters"

This reverts commit a34f324.

* update snapshot
legrego added a commit that referenced this pull request Sep 5, 2019
* ensure example url wraps properly when the space identifier is long

* restrict space identifier length to 500 characters

* Revert "restrict space identifier length to 500 characters"

This reverts commit a34f324.

* update snapshot
patrykkopycinski pushed a commit to patrykkopycinski/kibana that referenced this pull request May 6, 2026
…44871)

* ensure example url wraps properly when the space identifier is long

* restrict space identifier length to 500 characters

* Revert "restrict space identifier length to 500 characters"

This reverts commit a34f324.

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

Labels

Feature:Security/Spaces Platform Security - Spaces feature release_note:fix Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t// v7.4.0 v7.5.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Limit the number of characters while creating a space

3 participants