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

Support comments in JSON request body #599

Open
ashitaprasad opened this issue Feb 22, 2025 · 2 comments · May be fixed by #623
Open

Support comments in JSON request body #599

ashitaprasad opened this issue Feb 22, 2025 · 2 comments · May be fixed by #623
Labels
good first issue Good for newcomers

Comments

@ashitaprasad
Copy link
Member

Tell us about the task you want to perform and are unable to do so because the feature is not available

Support comments in JSON request body

{
  // one-line comment
  "foo": "bar",
  /*
   * multi-line block comment
   */
  "baz": "qux",
}

Potential solution can be searched here - https://fluttergems.dev/json/

@ashitaprasad ashitaprasad added enhancement New feature or request FOSSHack2025 good first issue Good for newcomers labels Feb 22, 2025
@FreeBono
Copy link
Contributor

Hello, @ashitaprasad. Can I take up this issue? I think Using regular expressions to solve this would be a good approach

@ashitaprasad
Copy link
Member Author

@FreeBono You can look into the link shared and check out some package first which has this feature.
Then you can use it directly or go through its code base to understand a good approach to do it.

FreeBono added a commit to FreeBono/apidash that referenced this issue Feb 25, 2025
FreeBono added a commit to FreeBono/apidash that referenced this issue Feb 25, 2025
@FreeBono FreeBono linked a pull request Feb 25, 2025 that will close this issue
9 tasks
@ashitaprasad ashitaprasad removed FOSSHack2025 enhancement New feature or request labels Feb 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants