Skip to content

Remove Manual Upgrade and Automatic Migration AJS #4407

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

Merged
merged 2 commits into from
Apr 6, 2023

Conversation

sarahrudy
Copy link
Contributor

@sarahrudy sarahrudy commented Mar 13, 2023

Proposed changes

  • Remove Manual Upgrade section because the flag to toggle Analytics.js on is no longer available in the customer's workspace
  • Remove Automatic Migration section because it mentions it in the yellow box directly above that so it seems redundant
  • Update section about reverting to AJS Classic to let customer know that it is no longer possible to revert to AJS Classic themselves (I intentionally didn't add anything about it being possible to revert to AJS Classic by request, because I worry this will encourage customers to use this as a temporary solution instead of upgrading to AJS 2.0 now)
  • Add sentence about how Cases That Require Additional intervention do not necessarily apply to every customer and remove reference to toggle

Merge timing

  • ASAP once approved?

- Remove Manual Upgrade section because the flag to toggle Analytics.js on is no longer available in the customer's workspace
- Remove Automatic Migration section because it mentions it in the yellow box directly above that so it seem redundant 
- Update section about reverting to AJS Classic to let customer know that it is no longer possible to revert to AJS Classic themselves (I intentionally didn't add anything about it being possible to revert to AJS Classic by request, because I worry this will encourage customers to use this as a temporary solution instead of upgrading to AJS 2.0 now)
- Add sentence about how Cases That Require Additional intervention do not necessarily apply to every customer and remove reference to toggle
@sarahrudy sarahrudy added the KCS label Mar 13, 2023
@stayseesong stayseesong self-requested a review April 5, 2023 16:09
@stayseesong stayseesong marked this pull request as ready for review April 6, 2023 16:46
@stayseesong stayseesong requested a review from a team as a code owner April 6, 2023 16:46
@stayseesong stayseesong merged commit 3c64f17 into develop Apr 6, 2023
@stayseesong stayseesong deleted the sarahrudy-patch-6 branch April 6, 2023 16:47
@github-actions
Copy link
Contributor

github-actions bot commented Apr 6, 2023

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants