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

[Stateful sidenav] Feedback button #195751

Merged
merged 10 commits into from
Oct 11, 2024

Conversation

sebelga
Copy link
Contributor

@sebelga sebelga commented Oct 10, 2024

In this PR I've added a feedback btn in the new solution side nav for non trial users.

As long as a deployment is in the trial dates, the button does not show. When the trial ends, if the deployment is still running, then the button appears.

The button can be dismissed, but the state is in sessionStorage. That means that it reappears when Kibana is opened in a new browser window or tab.

How to test

  • Add the following in your kibana.dev.yml
xpack.cloud.id: "ftr_fake_cloud_id:aGVsbG8uY29tOjQ0MyRFUzEyM2FiYyRrYm4xMjNhYmM="
xpack.cloud.base_url: "https://cloud.elastic.co"
  • Launch Kibana and setup a solution on the space
  • You should see the feedback button
  • Add the following to the kibana.dev.yml
# Any date in the future
xpack.cloud.trial_end_date: "2024-10-20T10:10:00Z"
  • Refresh the browser window
  • The button should not appear

Screenshots

Screenshot 2024-10-10 at 12 43 54 Screenshot 2024-10-10 at 12 44 38 Screenshot 2024-10-10 at 12 44 54
Screenshot 2024-10-10 at 12 44 08

Fixes https://github.com/elastic/kibana-team/issues/1135

@sebelga sebelga self-assigned this Oct 10, 2024
@sebelga sebelga added Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) Feature:Chrome Core's Chrome UI (sidenav, header, breadcrumbs) release_note:skip Skip the PR/issue when compiling release notes backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Oct 10, 2024
@sebelga sebelga marked this pull request as ready for review October 10, 2024 14:25
@sebelga sebelga requested a review from a team as a code owner October 10, 2024 14:25
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

Copy link
Contributor

@Dosant Dosant 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, I can test tomorrow

Copy link
Contributor

@Dosant Dosant left a comment

Choose a reason for hiding this comment

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

tested, works well

@sebelga sebelga enabled auto-merge (squash) October 11, 2024 10:16
@sebelga sebelga merged commit c1c70c7 into elastic:main Oct 11, 2024
28 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11294195504

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
navigation 76 77 +1
serverless 77 78 +1
total +2

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
@kbn/shared-ux-chrome-navigation 29 30 +1
navigation 54 55 +1
total +2

Async chunks

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

id before after diff
navigation 24.1KB 25.3KB +1.2KB
serverless 24.7KB 25.9KB +1.2KB
total +2.4KB

Page load bundle

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

id before after diff
core 449.4KB 449.8KB +366.0B
navigation 17.1KB 17.3KB +215.0B
total +581.0B
Unknown metric groups

API count

id before after diff
@kbn/shared-ux-chrome-navigation 38 39 +1
navigation 56 57 +1
total +2

History

cc @sebelga

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 11, 2024
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@sebelga sebelga deleted the stateful-sidenav/feedback-btn branch October 11, 2024 14:40
kibanamachine added a commit that referenced this pull request Oct 11, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[Stateful sidenav] Feedback button
(#195751)](#195751)

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

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

<!--BACKPORT [{"author":{"name":"Sébastien
Loix","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-11T14:29:39Z","message":"[Stateful
sidenav] Feedback button
(#195751)","sha":"c1c70c73aa02931f39045fdf2bc7938aea53c58d","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor","Feature:Chrome"],"title":"[Stateful
sidenav] Feedback
button","number":195751,"url":"https://github.com/elastic/kibana/pull/195751","mergeCommit":{"message":"[Stateful
sidenav] Feedback button
(#195751)","sha":"c1c70c73aa02931f39045fdf2bc7938aea53c58d"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195751","number":195751,"mergeCommit":{"message":"[Stateful
sidenav] Feedback button
(#195751)","sha":"c1c70c73aa02931f39045fdf2bc7938aea53c58d"}}]}]
BACKPORT-->

Co-authored-by: Sébastien Loix <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) Feature:Chrome Core's Chrome UI (sidenav, header, breadcrumbs) release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants