Migrate OrderSummary component to GraphQL
What does this MR do?
This MR is just about porting the existing component OrderSummary
from Vuex to GraphQL. Our goal is to re-create the current purchase flow with GraphQL instead of Vuex.
So, despite this MR is a little bit big it does just two things:
- Prepares the init step that the app uses to render the
OrderSummary
component. - Ports the
SubscriptionDetails
component to GraphQL so treat this MR as an MR that changes the backend of Vue.js components from Vuex to Apollo.
Additional notes:
- Yes, the code may be ugly in some places but this is the 1st MR out of the 4 or 5.
- All non-blocking suggestions will be addressed in the next MRs.
See this epic for more information.
Related: #321648 (closed)
How to test it?
Enable the corresponding feature flag: Feature.enable(:new_route_ci_minutes_purchase)
.
Screenshots (strongly suggested)
Not applicable. Everything stayed the same.
Does this MR meet the acceptance criteria?
Conformity
-
📋 Does this MR need a changelog?-
I have included a changelog entry. -
I have not included a changelog entry because _____.
-
-
Documentation (if required) -
Code review guidelines -
Merge request performance guidelines -
Style guides -
Database guides -
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. -
Tested in all supported browsers -
Informed Infrastructure department of a default or new setting change, if applicable per definition of done
Security
If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:
-
Label as security and @ mention @gitlab-com/gl-security/appsec
-
The MR includes necessary changes to maintain consistency between UI, API, email, or other methods -
Security reports checked/validated by a reviewer from the AppSec team
Edited by Vitaly Slobodin