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

GH-1777 and GH-1776 Plus Promo modals #458

Merged
merged 26 commits into from Oct 16, 2019
Merged

Commits on Sep 18, 2019

Commits on Sep 27, 2019

Commits on Oct 1, 2019

  1. Fix path bug in i18n-checker tool. Consolidate redundant entries in m…

    …essages. Add check icon to plus promo modal. Modify plus promo modal css.
    wlycdgr committed Oct 1, 2019

Commits on Oct 6, 2019

  1. Factor plus promo modal rendering out to shared component. Implement …

    …rendering in panel. Implement display context CSS adjustments.
    wlycdgr committed Oct 6, 2019

Commits on Oct 7, 2019

  1. Move Plus Promo modal rendering to a PlusPromoModal shared component.…

    … Begin to implement conditional panel display logic
    wlycdgr committed Oct 7, 2019
  2. Create ModalPromos background class responsible for modal promo relat…

    …ed business logic and state management
    wlycdgr committed Oct 7, 2019

Commits on Oct 8, 2019

  1. Add local state to Panel to make sure component rerenders after user …

    …dismisses promo panel. Send record of promo interaction to background
    wlycdgr committed Oct 8, 2019
  2. Factor PlusPromoModal rendering out to helper in Panel. Extend condit…

    …ional rendering logic to account for upgrade version of modal
    wlycdgr committed Oct 8, 2019
  3. Mark PromoModals methods as static. Refine PlusPromoModal implementat…

    …ion and add debug code
    wlycdgr committed Oct 8, 2019

Commits on Oct 16, 2019

  1. Pull non-shared upgrade version of plus promo modal out of shared Plu…

    …sPromoModal component and into Panel to simplify
    wlycdgr committed Oct 16, 2019