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

Resolve Discrepancies in Blaze Card Visibility with Legacy Jetpack Versions #11568

Closed
pmusolino opened this issue May 22, 2024 · 2 comments
Closed
Labels
feature: blaze Related to the Blaze project priority: low Not many people are affected or there’s a workaround, etc. type: bug A confirmed bug.

Comments

@pmusolino
Copy link
Member

pmusolino commented May 22, 2024

Description:

I've identified a critical issue affecting the visibility of the Blaze card, specifically when interacting with stores running on outdated Jetpack versions.

Issue Summary:

After the initial login to a store via the mobile app, the Blaze card displays as expected. However, upon relaunching the app, the Blaze card becomes hidden as if the store is no longer eligible to display it.

Investigation Findings:

  • The display logic for the Blaze card depends on multiple properties, including can_blaze.
  • The can_blaze property is expected from two endpoints: me/sites (post-login) and sites/:id (app startup).
  • On the first app launch, sites/:id isn't called since me/sites provides all necessary information. However, subsequent launches rely on sites/:id.
  • Testing revealed that sites/:id does not return the can_blaze property for sites with Jetpack versions older than 12.0.

Impact:
Users with Jetpack versions prior to 12.0 do not see the Blaze card in the app, potentially missing out on its features.

Potential Solutions:

  1. Use the can_blaze property from me/sites, store it locally, and rely on this stored value.
  2. Display an error on the Blaze card for users with Jetpack versions older than 12.0.
  3. Do not display the Blaze card at all but ensure the Jetpack version is checked to prevent discrepancies during the app's initial launch or store switching.

Preferred Solution:
The first solution is preferred, as it would allow users with older Jetpack versions to access the Blaze card features, considering that version 12.0 of Jetpack was released approximately a year ago.

Extra info here: pe5sF9-2Gh

@pmusolino pmusolino added type: bug A confirmed bug. feature: blaze Related to the Blaze project labels May 22, 2024
@dangermattic
Copy link
Collaborator

dangermattic commented May 22, 2024

Thanks for reporting! 👍

@JorgeMucientes JorgeMucientes added the priority: low Not many people are affected or there’s a workaround, etc. label Nov 20, 2024
@JorgeMucientes
Copy link
Contributor

Closing in favor of this other agreed solution to the problem: #11683

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: blaze Related to the Blaze project priority: low Not many people are affected or there’s a workaround, etc. type: bug A confirmed bug.
Projects
None yet
Development

No branches or pull requests

3 participants