Skip to content

Redirect endpoint#103899

Merged
streamich merged 30 commits intoelastic:masterfrom
streamich:redirect-endpoint-3
Jul 6, 2021
Merged

Redirect endpoint#103899
streamich merged 30 commits intoelastic:masterfrom
streamich:redirect-endpoint-3

Conversation

@streamich
Copy link
Copy Markdown
Contributor

@streamich streamich commented Jun 30, 2021

Summary

Closes #98112

Checklist

Delete any items that are not applicable to this PR.

Risk Matrix

Delete this section if it is not applicable to this PR.

Before closing this PR, invite QA, stakeholders, and other developers to identify risks that should be tested prior to the change/feature release.

When forming the risk matrix, consider some of the following examples and how they may potentially impact the change:

Risk Probability Severity Mitigation/Notes
... ... ... ...

For maintainers

@streamich streamich requested a review from a team as a code owner June 30, 2021 14:32
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app-services (Team:AppServices)

@streamich streamich added the release_note:feature Makes this part of the condensed release notes label Jun 30, 2021
@streamich
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@streamich streamich requested a review from a team as a code owner July 5, 2021 14:20
throw error;
}

const { state: migratedParams } = migrateToLatest(locator.migrations, {
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.

cc @stacey-gammon should we make migration happen internally inside navigate() ? one less thing that to worry about for the consumer, but not really. navigate would change and optionally accept version in. consumer would still need to make sure correct version is provided if the state is not of latest version. having to do them explicitly is kind of nice as you can check if anything changed in the state and warn the user about it.

Copy link
Copy Markdown
Contributor

@ppisljar ppisljar left a comment

Choose a reason for hiding this comment

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

code LGTM

@streamich streamich requested a review from a team as a code owner July 6, 2021 08:48
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
kibanaUtils 191 236 +45
share 55 66 +11
total +56

Public APIs missing comments

Total count of every public API that lacks a comment. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats comments for more detailed information.

id before after diff
kibanaUtils 365 356 -9
share 81 83 +2
total -7

Async chunks

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

id before after diff
kibanaUtils 108.3KB 105.8KB -2.6KB

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
share 7 8 +1

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kibanaUtils 136.9KB 171.8KB +34.9KB
share 74.8KB 83.1KB +8.3KB
total +43.2KB
Unknown metric groups

API count

id before after diff
kibanaUtils 543 549 +6
share 119 123 +4
total +10

History

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

@streamich streamich added the auto-backport Deprecated - use backport:version if exact versions are needed label Jul 6, 2021
@streamich streamich merged commit ec5d398 into elastic:master Jul 6, 2021
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Jul 6, 2021
* feat: 🎸 add redirect endpoint app

* feat: 🎸 improve spinner design

* feat: 🎸 implement basic version of redirect endpoint

* feat: 🎸 render errors for user on the screen

* feat: 🎸 improve error message display

* feat: 🎸 improve error display

* feat: 🎸 improve locator errors

* feat: 🎸 improve errors

* feat: 🎸 improve persistable state types

* feat: 🎸 implement migrateToLatest function

* feat: 🎸 migrate locator params to the latest in redirect endp

* Update src/plugins/kibana_utils/common/persistable_state/migrate_to_latest.ts

* refactor: 💡 make Versioned state be an object

* fix: 🐛 use new VersionedState interface in redirect endpoint

* refactor: 💡 move parseSearchParams into a separate function

* feat: 🎸 implement redirect URL formatter

* feat: 🎸 export redirect URL parsing and formatting functions

* refactor: 💡 use relative import

* test: 💍 add example links through redirect endpoint

* test: 💍 use updated VersionedState type

* test: 💍 add redirect manager tests

* feat: 🎸 add redirect endpoint app to common schema

* chore: 🤖 update telemetry schema

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine
Copy link
Copy Markdown
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 Jul 6, 2021
* feat: 🎸 add redirect endpoint app

* feat: 🎸 improve spinner design

* feat: 🎸 implement basic version of redirect endpoint

* feat: 🎸 render errors for user on the screen

* feat: 🎸 improve error message display

* feat: 🎸 improve error display

* feat: 🎸 improve locator errors

* feat: 🎸 improve errors

* feat: 🎸 improve persistable state types

* feat: 🎸 implement migrateToLatest function

* feat: 🎸 migrate locator params to the latest in redirect endp

* Update src/plugins/kibana_utils/common/persistable_state/migrate_to_latest.ts

* refactor: 💡 make Versioned state be an object

* fix: 🐛 use new VersionedState interface in redirect endpoint

* refactor: 💡 move parseSearchParams into a separate function

* feat: 🎸 implement redirect URL formatter

* feat: 🎸 export redirect URL parsing and formatting functions

* refactor: 💡 use relative import

* test: 💍 add example links through redirect endpoint

* test: 💍 use updated VersionedState type

* test: 💍 add redirect manager tests

* feat: 🎸 add redirect endpoint app to common schema

* chore: 🤖 update telemetry schema

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Vadim Dalecky <streamich@gmail.com>
@bhavyarm
Copy link
Copy Markdown
Contributor

How do I test this pease? Thanks!

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 Feature:SharingURLs Short URLs and Share URL features release_note:feature Makes this part of the condensed release notes review v7.15.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

URL Service redirect endpoint

7 participants