refactor: mfe_context response to serialize object keys to camelcase #31930
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
shahbaz-shabbir05
force-pushed
the
shahbaz/VAN-1311
branch
from
250c87c
to
3891ac8
Compare
syedsajjadkazmii
requested changes
Mar 17, 2023
shahbaz-shabbir05
force-pushed
the
shahbaz/VAN-1311
branch
from
8b401dc
to
7d79c18
Compare
shahbaz-shabbir05
force-pushed
the
shahbaz/VAN-1311
branch
6 times, most recently
from
e687c85
to
d52fc24
Compare
shahbaz-shabbir05
force-pushed
the
shahbaz/VAN-1311
branch
2 times, most recently
from
cdfa5c3
to
292c07e
Compare
syedsajjadkazmii
approved these changes
Apr 6, 2023
shahbaz-shabbir05
force-pushed
the
shahbaz/VAN-1311
branch
from
10f27d1
to
97d94f3
Compare
shahbaz-shabbir05
force-pushed
the
shahbaz/VAN-1311
branch
from
97d94f3
to
ab61e66
Compare
shahbaz-shabbir05
force-pushed
the
shahbaz/VAN-1311
branch
from
ab61e66
to
87d76d9
Compare
|
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
|
EdX Release Notice: This PR has been deployed to the production environment. |
|
EdX Release Notice: This PR has been deployed to the staging environment in preparation for a release to production. |
|
EdX Release Notice: This PR has been deployed to the production environment. |
mubbsharanwar
pushed a commit
that referenced
this pull request
Aug 17, 2023
mubbsharanwar
added a commit
that referenced
this pull request
Aug 22, 2023
* refactor: mfe_context response to serialize object keys to camelcase (#31930) * test: add tests for MFE Context API serializser (#32179) * fix: empty pipelineUserDetail object pipelineUserDetail object should be empty when pipeline is not running --------- Co-authored-by: Shahbaz Shabbir <32649010+shahbaz-arbisoft@users.noreply.github.com>
shURenZHOUluxun
pushed a commit
to EduTrigger/edx-platform
that referenced
this pull request
Jan 3, 2024
* refactor: mfe_context response to serialize object keys to camelcase (openedx#31930) * test: add tests for MFE Context API serializser (openedx#32179) * fix: empty pipelineUserDetail object pipelineUserDetail object should be empty when pipeline is not running --------- Co-authored-by: Shahbaz Shabbir <32649010+shahbaz-arbisoft@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request modifies the serialization of the back-end response by converting the keys of objects to camelCase. By making this change, we can avoid the need to convert the keys to camelCase on the front-end and ensures that all related unit tests have been updated accordingly.
In order to prevent compatibility issues, it is necessary to remove any front-end code that converts keys from snake_case to camelCase format where it is required.
Ticket: VAN-1311