-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Migrate test plugins ⇒ NP (kbn_tp_top_nav) #60092
Migrate test plugins ⇒ NP (kbn_tp_top_nav) #60092
Conversation
@@ -0,0 +1,15 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lizozom just want to know your opinion. should we provide tsconfig.json
and package.json
for test/plugin_functional/plugins/**
plugins?
it looks a little odd to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, if things work without them, I guess no :) @alexwizp
public setup(core: CoreSetup, { navigation }: { navigation: NavigationPublicPluginSetup }) { | ||
const customExtension = { | ||
id: 'registered-prop', | ||
label: 'Registered Button', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From my point of view all static string values should be localized
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@elasticmachine merge upstream |
Pinging @elastic/kibana-app-arch (Team:AppArch) |
@lizozom could you please have a look |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Migrated top nav plugin test to the new platform * Removed old plugin. Some refactoring. Co-authored-by: Elastic Machine <[email protected]>
* Migrated top nav plugin test to the new platform * Removed old plugin. Some refactoring. Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
Part of #51275
Summary
Checklist
Delete any items that are not applicable to this PR.
For maintainers