Skip to content

Conversation

@we-chintan-p001
Copy link
Collaborator

Overview

  • I have performed a self-review of my code
  • manual testing has been done
  • The code modified as part of this PR has been covered with tests

Problem statement

Handle Snake_case Response From API and also Handle lowerCamelCase Request Data

Solution

Added _parseKeys method in API which can parse snake_case to lowerCamelCase and vice versa. The response is parsed when we receive it and then it can be used by callback or ajax method caller. Also, before sending data it parsed and that parsed data is sent in requests.

Dependencies

No dependencies were added.

Links

Task

Videos:
Parsing Request Data: https://youtu.be/HThTvH7TFHs
Parsing Response Data: https://youtu.be/f618tKk7fHY

we-chintan-p001 and others added 6 commits October 28, 2022 20:23
Need to handle array of dynamic items, and instead of converting string to uppercase, needs to convert from snake_case
all data types are supported: list, map, bool, string, List containing map, map containing list, list containing lists, map containing maps.
@themodernpk themodernpk merged commit ee43365 into develop Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants