fix axios template: [object Object] is not valid JSON.#5283
Merged
RicoSuter merged 2 commits intoRicoSuter:masterfrom Feb 27, 2026
Merged
fix axios template: [object Object] is not valid JSON.#5283RicoSuter merged 2 commits intoRicoSuter:masterfrom
RicoSuter merged 2 commits intoRicoSuter:masterfrom
Conversation
Fixes RicoSuter#4973 Added condition to check if template is `Framework.IsAxios` then return data without parse, cause axios convert json string to data object by default.
Owner
|
can you maybe update with latest master, hopefully CI goes green |
Contributor
Author
|
@RicoSuter Done. Now CI is waiting for approval. |
This was referenced Apr 8, 2026
Merged
Closed
Closed
Closed
Merged
This was referenced Apr 13, 2026
Closed
github-actions bot
pushed a commit
to EelcoLos/nx-tinkering
that referenced
this pull request
Apr 14, 2026
Updated [NSwag.Core](https://github.com/RicoSuter/NSwag) from 14.6.3 to 14.7.0. <details> <summary>Release notes</summary> _Sourced from [NSwag.Core's releases](https://github.com/RicoSuter/NSwag/releases)._ ## 14.7.0 ## What's Changed * Upgrade to NUKE 10 by @lahma in RicoSuter/NSwag#5310 * Upgrade to NUKE 10.1.0 by @lahma in RicoSuter/NSwag#5314 * Fix specifying runtime for `NSwag.Npm` resulting in a `NConsole.UnusedArgumentException` by @ptasev in RicoSuter/NSwag#5320 * fix: nullpointer by @MeikelLP in RicoSuter/NSwag#5337 * fix axios template: [object Object] is not valid JSON. by @adnanalbeda in RicoSuter/NSwag#5283 * Fix legacy document transformation by @copyleftproducts in RicoSuter/NSwag#5315 * Fix compile error in generated C# client when media type contains quotes by @bkoelman in RicoSuter/NSwag#5345 * Fix duplicate client/operation names when operation IDs contain multiple underscores by @Copilot in RicoSuter/NSwag#5348 * Update to NJsonSchema v11.6.0 and Namotion.Reflection v3.5.0 (v14.7.0) by @RicoSuter in RicoSuter/NSwag#5357 ## NJsonSchema v11.6.0 (potentially breaking changes) This release updates to [NJsonSchema v11.6.0](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0) and [Namotion.Reflection v3.5.0](https://github.com/RicoSuter/Namotion.Reflection/commits/master/), which include the following changes that may affect generated code: - **C# 11 `required` keyword now correctly recognized**: Properties using the C# 11 `required` keyword (via `RequiredMemberAttribute` / `JsonRequiredAttribute`) are now properly treated as required in the schema and generated code. Previously these were incorrectly treated as optional. - **Removed extra blank line** before class declarations in generated C# controller code (cosmetic). - New `WriteAccessor` setting to control property setter syntax (`set` vs `init`). - New `JsonLibraryVersion` setting for controlling enum attribute generation with System.Text.Json. - Fixed `MinLength` validation no longer incorrectly applied to DateTime properties. - Fixed nullable enum array detection for string enum converters. - Support for public fields with System.Text.Json when `IncludeFields` is enabled. For the full list of changes, see the [NJsonSchema v11.6.0 release notes](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0). ## New Contributors * @ptasev made their first contribution in RicoSuter/NSwag#5320 * @MeikelLP made their first contribution in RicoSuter/NSwag#5337 * @adnanalbeda made their first contribution in RicoSuter/NSwag#5283 * @copyleftproducts made their first contribution in RicoSuter/NSwag#5315 * @Copilot made their first contribution in RicoSuter/NSwag#5348 **Full Changelog**: RicoSuter/NSwag@v14.6.3...v14.7.0 Commits viewable in [compare view](RicoSuter/NSwag@v14.6.3...v14.7.0). </details> Updated [NSwag.Core.Yaml](https://github.com/RicoSuter/NSwag) from 14.6.3 to 14.7.0. <details> <summary>Release notes</summary> _Sourced from [NSwag.Core.Yaml's releases](https://github.com/RicoSuter/NSwag/releases)._ ## 14.7.0 ## What's Changed * Upgrade to NUKE 10 by @lahma in RicoSuter/NSwag#5310 * Upgrade to NUKE 10.1.0 by @lahma in RicoSuter/NSwag#5314 * Fix specifying runtime for `NSwag.Npm` resulting in a `NConsole.UnusedArgumentException` by @ptasev in RicoSuter/NSwag#5320 * fix: nullpointer by @MeikelLP in RicoSuter/NSwag#5337 * fix axios template: [object Object] is not valid JSON. by @adnanalbeda in RicoSuter/NSwag#5283 * Fix legacy document transformation by @copyleftproducts in RicoSuter/NSwag#5315 * Fix compile error in generated C# client when media type contains quotes by @bkoelman in RicoSuter/NSwag#5345 * Fix duplicate client/operation names when operation IDs contain multiple underscores by @Copilot in RicoSuter/NSwag#5348 * Update to NJsonSchema v11.6.0 and Namotion.Reflection v3.5.0 (v14.7.0) by @RicoSuter in RicoSuter/NSwag#5357 ## NJsonSchema v11.6.0 (potentially breaking changes) This release updates to [NJsonSchema v11.6.0](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0) and [Namotion.Reflection v3.5.0](https://github.com/RicoSuter/Namotion.Reflection/commits/master/), which include the following changes that may affect generated code: - **C# 11 `required` keyword now correctly recognized**: Properties using the C# 11 `required` keyword (via `RequiredMemberAttribute` / `JsonRequiredAttribute`) are now properly treated as required in the schema and generated code. Previously these were incorrectly treated as optional. - **Removed extra blank line** before class declarations in generated C# controller code (cosmetic). - New `WriteAccessor` setting to control property setter syntax (`set` vs `init`). - New `JsonLibraryVersion` setting for controlling enum attribute generation with System.Text.Json. - Fixed `MinLength` validation no longer incorrectly applied to DateTime properties. - Fixed nullable enum array detection for string enum converters. - Support for public fields with System.Text.Json when `IncludeFields` is enabled. For the full list of changes, see the [NJsonSchema v11.6.0 release notes](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0). ## New Contributors * @ptasev made their first contribution in RicoSuter/NSwag#5320 * @MeikelLP made their first contribution in RicoSuter/NSwag#5337 * @adnanalbeda made their first contribution in RicoSuter/NSwag#5283 * @copyleftproducts made their first contribution in RicoSuter/NSwag#5315 * @Copilot made their first contribution in RicoSuter/NSwag#5348 **Full Changelog**: RicoSuter/NSwag@v14.6.3...v14.7.0 Commits viewable in [compare view](RicoSuter/NSwag@v14.6.3...v14.7.0). </details> Updated [NSwag.Generation.AspNetCore](https://github.com/RicoSuter/NSwag) from 14.6.3 to 14.7.0. <details> <summary>Release notes</summary> _Sourced from [NSwag.Generation.AspNetCore's releases](https://github.com/RicoSuter/NSwag/releases)._ ## 14.7.0 ## What's Changed * Upgrade to NUKE 10 by @lahma in RicoSuter/NSwag#5310 * Upgrade to NUKE 10.1.0 by @lahma in RicoSuter/NSwag#5314 * Fix specifying runtime for `NSwag.Npm` resulting in a `NConsole.UnusedArgumentException` by @ptasev in RicoSuter/NSwag#5320 * fix: nullpointer by @MeikelLP in RicoSuter/NSwag#5337 * fix axios template: [object Object] is not valid JSON. by @adnanalbeda in RicoSuter/NSwag#5283 * Fix legacy document transformation by @copyleftproducts in RicoSuter/NSwag#5315 * Fix compile error in generated C# client when media type contains quotes by @bkoelman in RicoSuter/NSwag#5345 * Fix duplicate client/operation names when operation IDs contain multiple underscores by @Copilot in RicoSuter/NSwag#5348 * Update to NJsonSchema v11.6.0 and Namotion.Reflection v3.5.0 (v14.7.0) by @RicoSuter in RicoSuter/NSwag#5357 ## NJsonSchema v11.6.0 (potentially breaking changes) This release updates to [NJsonSchema v11.6.0](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0) and [Namotion.Reflection v3.5.0](https://github.com/RicoSuter/Namotion.Reflection/commits/master/), which include the following changes that may affect generated code: - **C# 11 `required` keyword now correctly recognized**: Properties using the C# 11 `required` keyword (via `RequiredMemberAttribute` / `JsonRequiredAttribute`) are now properly treated as required in the schema and generated code. Previously these were incorrectly treated as optional. - **Removed extra blank line** before class declarations in generated C# controller code (cosmetic). - New `WriteAccessor` setting to control property setter syntax (`set` vs `init`). - New `JsonLibraryVersion` setting for controlling enum attribute generation with System.Text.Json. - Fixed `MinLength` validation no longer incorrectly applied to DateTime properties. - Fixed nullable enum array detection for string enum converters. - Support for public fields with System.Text.Json when `IncludeFields` is enabled. For the full list of changes, see the [NJsonSchema v11.6.0 release notes](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0). ## New Contributors * @ptasev made their first contribution in RicoSuter/NSwag#5320 * @MeikelLP made their first contribution in RicoSuter/NSwag#5337 * @adnanalbeda made their first contribution in RicoSuter/NSwag#5283 * @copyleftproducts made their first contribution in RicoSuter/NSwag#5315 * @Copilot made their first contribution in RicoSuter/NSwag#5348 **Full Changelog**: RicoSuter/NSwag@v14.6.3...v14.7.0 Commits viewable in [compare view](RicoSuter/NSwag@v14.6.3...v14.7.0). </details> Updated [NSwag.MSBuild](https://github.com/RicoSuter/NSwag) from 14.6.3 to 14.7.0. <details> <summary>Release notes</summary> _Sourced from [NSwag.MSBuild's releases](https://github.com/RicoSuter/NSwag/releases)._ ## 14.7.0 ## What's Changed * Upgrade to NUKE 10 by @lahma in RicoSuter/NSwag#5310 * Upgrade to NUKE 10.1.0 by @lahma in RicoSuter/NSwag#5314 * Fix specifying runtime for `NSwag.Npm` resulting in a `NConsole.UnusedArgumentException` by @ptasev in RicoSuter/NSwag#5320 * fix: nullpointer by @MeikelLP in RicoSuter/NSwag#5337 * fix axios template: [object Object] is not valid JSON. by @adnanalbeda in RicoSuter/NSwag#5283 * Fix legacy document transformation by @copyleftproducts in RicoSuter/NSwag#5315 * Fix compile error in generated C# client when media type contains quotes by @bkoelman in RicoSuter/NSwag#5345 * Fix duplicate client/operation names when operation IDs contain multiple underscores by @Copilot in RicoSuter/NSwag#5348 * Update to NJsonSchema v11.6.0 and Namotion.Reflection v3.5.0 (v14.7.0) by @RicoSuter in RicoSuter/NSwag#5357 ## NJsonSchema v11.6.0 (potentially breaking changes) This release updates to [NJsonSchema v11.6.0](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0) and [Namotion.Reflection v3.5.0](https://github.com/RicoSuter/Namotion.Reflection/commits/master/), which include the following changes that may affect generated code: - **C# 11 `required` keyword now correctly recognized**: Properties using the C# 11 `required` keyword (via `RequiredMemberAttribute` / `JsonRequiredAttribute`) are now properly treated as required in the schema and generated code. Previously these were incorrectly treated as optional. - **Removed extra blank line** before class declarations in generated C# controller code (cosmetic). - New `WriteAccessor` setting to control property setter syntax (`set` vs `init`). - New `JsonLibraryVersion` setting for controlling enum attribute generation with System.Text.Json. - Fixed `MinLength` validation no longer incorrectly applied to DateTime properties. - Fixed nullable enum array detection for string enum converters. - Support for public fields with System.Text.Json when `IncludeFields` is enabled. For the full list of changes, see the [NJsonSchema v11.6.0 release notes](https://github.com/RicoSuter/NJsonSchema/releases/tag/v11.6.0). ## New Contributors * @ptasev made their first contribution in RicoSuter/NSwag#5320 * @MeikelLP made their first contribution in RicoSuter/NSwag#5337 * @adnanalbeda made their first contribution in RicoSuter/NSwag#5283 * @copyleftproducts made their first contribution in RicoSuter/NSwag#5315 * @Copilot made their first contribution in RicoSuter/NSwag#5348 **Full Changelog**: RicoSuter/NSwag@v14.6.3...v14.7.0 Commits viewable in [compare view](RicoSuter/NSwag@v14.6.3...v14.7.0). </details> Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore <dependency name> major version` will close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself) - `@dependabot ignore <dependency name> minor version` will close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself) - `@dependabot ignore <dependency name>` will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself) - `@dependabot unignore <dependency name>` will remove all of the ignore conditions of the specified dependency - `@dependabot unignore <dependency name> <ignore condition>` will remove the ignore condition of the specified dependency and ignore conditions </details> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Eelco Los <5102501+EelcoLos@users.noreply.github.com>
This was referenced Apr 14, 2026
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Fixes #4973
Added condition to check if template is
Framework.IsAxiosthen return data without parse, cause axios convert json string to data object by default.