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

Regression: Adjusted departments upsell modal message #28258

Merged
merged 4 commits into from Mar 6, 2023

Conversation

aleksandernsilva
Copy link
Contributor

@aleksandernsilva aleksandernsilva commented Mar 3, 2023

Proposed changes (including videos or screenshots)

This PR adjusts the logic to display the correct message on the departments upsell modal. This affects users with a trial license, expired expired licence, as well as no licence.

Issue(s)

Steps to test or reproduce

Further comments

OC-712

@aleksandernsilva aleksandernsilva self-assigned this Mar 3, 2023
@aleksandernsilva aleksandernsilva changed the title [FIX] Adjusted departments upsell modal message Regression: Adjusted departments upsell modal message Mar 3, 2023
@aleksandernsilva aleksandernsilva marked this pull request as ready for review March 3, 2023 15:29
@aleksandernsilva aleksandernsilva requested a review from a team as a code owner March 3, 2023 15:30
filipemarins
filipemarins previously approved these changes Mar 3, 2023
@codecov
Copy link

codecov bot commented Mar 3, 2023

Codecov Report

Merging #28258 (24881a6) into develop (0e969e9) will decrease coverage by 1.16%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28258      +/-   ##
===========================================
- Coverage    44.92%   43.77%   -1.16%     
===========================================
  Files          771      761      -10     
  Lines        14984    14769     -215     
  Branches      2091     2076      -15     
===========================================
- Hits          6732     6465     -267     
- Misses        7954     8030      +76     
+ Partials       298      274      -24     
Flag Coverage Δ
e2e 43.73% <0.00%> (-1.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@KevLehman KevLehman added the stat: ready to merge PR tested and approved waiting for merge label Mar 6, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 6, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 6, 2023
@kodiakhq kodiakhq bot merged commit 4d22ed0 into develop Mar 6, 2023
32 of 33 checks passed
@kodiakhq kodiakhq bot deleted the fix/departments-upsell-modal-message branch March 6, 2023 14:52
@sampaiodiego sampaiodiego mentioned this pull request Mar 9, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants