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

Rename apm_oss (#37) #72

Merged
merged 1 commit into from
Mar 8, 2021
Merged

Rename apm_oss (#37) #72

merged 1 commit into from
Mar 8, 2021

Conversation

boktorbb
Copy link
Contributor

@boktorbb boktorbb commented Mar 5, 2021

Signed-off-by: Bishoy Boktor [email protected]

Signed-off-by: Bishoy Boktor <[email protected]>
Copy link
Member

@kavilla kavilla left a comment

Choose a reason for hiding this comment

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

Just gave a comment that would should align upon but that's not blocking. LGTM!

@@ -69,7 +69,7 @@ export function onPremInstructions({
}),
message: i18n.translate('apmOss.tutorial.apmServer.callOut.message', {
defaultMessage: `Please make sure your APM Server is updated to 7.0 or higher. \
You can also migrate your 6.x data with the migration assistant found in Kibana's management section.`,
You can also migrate your 6.x data with the migration assistant found in OpenSearch Dashboards's management section.`,
Copy link
Member

Choose a reason for hiding this comment

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

I can't remember if I did OpenSearch Dashboards's or OpenSearch Dashboards' should we just leave it as Dashboards's

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've been leaving them as Dashboards's even though Dashboards' is the grammatically correct word. I don't think it matters too much though

@mihirsoni mihirsoni merged commit d4dabcb into opensearch-project:dashboards Mar 8, 2021
kavilla referenced this pull request in kavilla/OpenSearch-Dashboards Mar 19, 2021
Signed-off-by: Bishoy Boktor <[email protected]>
kavilla pushed a commit that referenced this pull request Mar 20, 2021
Signed-off-by: Bishoy Boktor <[email protected]>
mihirsoni pushed a commit that referenced this pull request Mar 20, 2021
Signed-off-by: Bishoy Boktor <[email protected]>
raintygao pushed a commit to raintygao/OpenSearch-Dashboards that referenced this pull request Aug 4, 2023
* feat: remove featureGroup and update management category

Signed-off-by: Lin Wang <[email protected]>

* feat: filter out empty feature category

Signed-off-by: Lin Wang <[email protected]>

---------

Signed-off-by: Lin Wang <[email protected]>
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Sep 14, 2023
* feat: remove featureGroup and update management category

Signed-off-by: Lin Wang <[email protected]>

* feat: filter out empty feature category

Signed-off-by: Lin Wang <[email protected]>

---------

Signed-off-by: Lin Wang <[email protected]>
Hailong-am pushed a commit to Hailong-am/OpenSearch-Dashboards that referenced this pull request Sep 15, 2023
* feat: remove featureGroup and update management category

Signed-off-by: Lin Wang <[email protected]>

* feat: filter out empty feature category

Signed-off-by: Lin Wang <[email protected]>

---------

Signed-off-by: Lin Wang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants