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

[Workplace Search] Fix personal dashboard logout link on local dev #102451

Merged
merged 1 commit into from
Jun 17, 2021

Conversation

cee-chen
Copy link
Member

@cee-chen cee-chen commented Jun 17, 2021

Summary

The logout link on the Personal Dashboard wasn't taking in the full Kibana URL into account and was broken on local due to the link URL missing the /xyz dev randomly generated base path.

logout

With this PR, logout links should work as expected in dev mode (EuiButtonTo takes care of correct Kibana URL generation for us).

Checklist

  • Works on local /xyz/ dev
  • Works in spaces
  • Works in non-dev mode

@cee-chen cee-chen added bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes v7.14.0 auto-backport Deprecated - use backport:version if exact versions are needed labels Jun 17, 2021
@cee-chen cee-chen requested review from yakhinvadim and a team June 17, 2021 06:29
@kibanamachine
Copy link
Contributor

💚 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
enterpriseSearch 2.1MB 2.1MB -6.0B

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

Copy link
Contributor

@yakhinvadim yakhinvadim left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this! 🙏

@cee-chen cee-chen merged commit e8c16e3 into elastic:master Jun 17, 2021
@cee-chen cee-chen deleted the ws-fix-logout branch June 17, 2021 16:08
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 17, 2021
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Jun 17, 2021
Co-authored-by: Constance <constancecchen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes v7.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants