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.

Sign up for GitHub

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

Jump to bottom

[Inventory] Update new Inventory empty state to show it is tech preview #196264

Closed
roshan-elastic opened this issue Oct 15, 2024 · 4 comments · Fixed by #196353
Closed

[Inventory] Update new Inventory empty state to show it is tech preview #196264

roshan-elastic opened this issue Oct 15, 2024 · 4 comments · Fixed by #196353
Assignees
@kpatticha
Labels
bug Fixes for quality problems that affect the customer experience Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team

Comments

@roshan-elastic
Copy link

roshan-elastic commented Oct 15, 2024 •
edited
Loading

Description

As the entityCentricExperience feature flag is on by default, users who visit observability will not see that the new inventory is in technical preview. This risks users where the cluster admin does not turn it off starting to use the feature without realising it is not GA functionality and being dissatisfied:

Mock
Image

Note : The left-hand nav does not support technical preview icons in some cases - as per the screenshot above

This issue is to ensure the empty state has a clear technical preview status and that the inventory wording should also point through to the inventory docs:

✔️ Acceptance criteria

1. Must Have

Must be delivered in this issue in order for the release to be valuable

Name Description
The empty state for inventory should be clearly marked as technical preview You can use standard EUI templates for this
The empty state for inventory should link through to the inventory docs Link the word 'inventory' to https://ela.st/docs-entity-inventory

4. Will Not Have (for now)

Explicitly will not be looked at within this issue

Name Description
- -
@roshan-elastic roshan-elastic added the needs-refinement A reason and acceptance criteria need to be defined for this issue label Oct 15, 2024
@botelastic botelastic bot added the needs-team Issues missing a team label label Oct 15, 2024
@roshan-elastic roshan-elastic added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Oct 15, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Oct 15, 2024
@roshan-elastic roshan-elastic changed the title Update new Inventory to show it is tech preview [Inventory] Update new Inventory to show it is tech preview Oct 15, 2024
@roshan-elastic roshan-elastic added the bug Fixes for quality problems that affect the customer experience label Oct 15, 2024
@roshan-elastic roshan-elastic changed the title [Inventory] Update new Inventory to show it is tech preview [Inventory] Update new Inventory empty state to show it is tech preview Oct 15, 2024
@roshan-elastic
Copy link
Author

Feel free to ask @kkurstak for support on this but perhaps there is a very simple 2 minute method we usually mark tech preview features in empty states?

@kpatticha
Copy link
Contributor

we can't show the technical preview icon in the side nav for stateful classic and serverless .

We can add the technical preview icon in the button "Enable" or the right side.

@roshan-elastic
Copy link
Author

Perfect @kpatticha - that'll work.

Quick Q - I've realised we don't link through to any docs that explain what the inventory actually is - I've updated the AC to include adding a link through to the docs in the existing wording if that's OK?

@kpatticha kpatticha self-assigned this Oct 15, 2024
@kpatticha kpatticha removed the needs-refinement A reason and acceptance criteria need to be defined for this issue label Oct 15, 2024
@kpatticha kpatticha mentioned this issue Oct 15, 2024
Merged
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Oct 16, 2024
@kpatticha
[Inventory] Add Technical preview icon and link to docs (elastic#196353)
4e875b8 
## Summary

closes elastic#196264

https://github.com/user-attachments/assets/a8fdf896-5d55-4060-a274-d2a4188c62b2
(cherry picked from commit 4ac575b)
kibanamachine added a commit that referenced this issue Oct 16, 2024
@kibanamachine @kpatticha
[8.x] [Inventory] Add Technical preview icon and link to docs (#196353)…
2b3ff67 
… (#196543)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Inventory] Add Technical preview icon and link to docs
(#196353)](#196353)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Katerina","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-16T12:16:05Z","message":"[Inventory]
Add Technical preview icon and link to docs (#196353)\n\n##
Summary\r\n\r\ncloses #196264
\r\n\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/a8fdf896-5d55-4060-a274-d2a4188c62b2","sha":"4ac575b3b291743af5fcd70f795ca43ad805fbe7","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","ci:project-deploy-observability","v8.16.0"],"title":"[Inventory]
Add Technical preview icon and link to
docs","number":196353,"url":"https://github.com/elastic/kibana/pull/196353","mergeCommit":{"message":"[Inventory]
Add Technical preview icon and link to docs (#196353)\n\n##
Summary\r\n\r\ncloses #196264
\r\n\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/a8fdf896-5d55-4060-a274-d2a4188c62b2","sha":"4ac575b3b291743af5fcd70f795ca43ad805fbe7"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196353","number":196353,"mergeCommit":{"message":"[Inventory]
Add Technical preview icon and link to docs (#196353)\n\n##
Summary\r\n\r\ncloses #196264
\r\n\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/a8fdf896-5d55-4060-a274-d2a4188c62b2","sha":"4ac575b3b291743af5fcd70f795ca43ad805fbe7"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Katerina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kpatticha kpatticha

Labels
bug Fixes for quality problems that affect the customer experience Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Inventory] Add Technical preview icon and link to docs kpatticha/kibana
3 participants
@kpatticha @elasticmachine @roshan-elastic