-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Alerting] only show trial upgrade when running with basic license #64865
Conversation
resolves elastic#64245 Prior to this PR, the "Upgrade your license" banner in the connectors list was displayed for gold licenses because the Service Now action requires platinum, and the check only looked for any actions disabled by license. Rather than display a different message for gold users, this PR changes the banner display logic to check for any actions disabled by license that also have a minimum required license of gold. That means gold+ users won't see the message, even for actions with a minimum required license of platinum+. Another perk of the gold license! This will continue to display the banner for basic users, but will no longer display it for gold users. It also continues to not display it for trial, platinum and enterprise users.
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
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
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 👍 I would confirm with @alexfrancoeur or @arisonl that the banner would only show for non-paying users (gold users won't see it anymore).
The "Get more actions" link still shows for gold users but I guess it's not aggressive enough to remove.
const hasActionsDisabledByLicense = availableActionTypes.some( | ||
action => !index[action.id].enabledInLicense | ||
// determine if there are actions disabled by license that that | ||
// would be enabled by upgrading to trial |
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.
nit: I would change the comment from trial
to a gold license
to be more clear on the logic below. Trial unlocks all license levels.
// would be enabled by upgrading to trial | |
// would be enabled by upgrading to a gold license |
@@ -136,6 +136,100 @@ describe('connector_add_flyout', () => { | |||
`"https://www.elastic.co/subscriptions"` | |||
); | |||
}); | |||
|
|||
it('does not render banner with subscription links when only platinum features are disabled due to licensing ', () => { |
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.
Unit tests ❤️
@elasticmachine merge upstream |
from #64865 (review)
Confirmed. Noted that we may want to enhance this later to show a more "subtle" experience for gold+ users, in case there are actions that require an upgraded license given the current gold+ license, but for now, not showing it for gold+ is fine. |
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
…lastic#64865) resolves elastic#64245 Prior to this PR, the "Upgrade your license" banner in the connectors list was displayed for gold licenses because the Service Now action requires platinum, and the check only looked for any actions disabled by license. Rather than display a different message for gold users, this PR changes the banner display logic to check for any actions disabled by license that also have a minimum required license of gold. That means gold+ users won't see the message, even for actions with a minimum required license of platinum+. Another perk of the gold license! This will continue to display the banner for basic users, but will no longer display it for gold users. It also continues to not display it for trial, platinum and enterprise users.
…64865) (#65138) resolves #64245 Prior to this PR, the "Upgrade your license" banner in the connectors list was displayed for gold licenses because the Service Now action requires platinum, and the check only looked for any actions disabled by license. Rather than display a different message for gold users, this PR changes the banner display logic to check for any actions disabled by license that also have a minimum required license of gold. That means gold+ users won't see the message, even for actions with a minimum required license of platinum+. Another perk of the gold license! This will continue to display the banner for basic users, but will no longer display it for gold users. It also continues to not display it for trial, platinum and enterprise users.
resolves #64245
Prior to this PR, the "Upgrade your license" banner in the connectors list
was displayed for gold licenses because the Service Now action requires
platinum, and the check only looked for any actions disabled by license.
Rather than display a different message for gold users, this PR changes the
banner display logic to check for any actions disabled by license that
also have a minimum required license of gold. That means gold+ users
won't see the message, even for actions with a minimum required license of
platinum+. Another perk of the gold license!
This will continue to display the banner for basic users, but will no longer
display it for gold users. It also continues to not display it for trial,
platinum and enterprise users.
Summary
Summarize your PR. If it involves visual changes include a screenshot or gif.
Checklist