Skip to content

fix(typescript-fetch): Use null as a value when the date value is nullable #21133

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tikki100
Copy link

@tikki100 tikki100 commented Apr 23, 2025

Currently, null is not used as a value for date values - even when the property is nullable. This means you're unable to construct a request with null. This PR fixes that.

PR checklist

  • Read the contribution guidelines.
  • [ x Pull Request title clearly describes the work in the pull request and Pull Request description provides details about how to validate the work. Missing information here may result in delayed response from the community.
  • Run the following to build the project and update samples:
    ./run-in-docker.sh generate -g typescript-fetch -o /gen/samples/client/petstore/typescript-fetch/builds/default-v3.0 -i modules/openapi-generator/src/test/resources/3_0/petstore-with-fake-endpoints-models-for-testing.yaml
    (For Windows users, please run the script in Git BASH)
    Commit all changed files.
    This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
    These must match the expectations made by your contribution.
    You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example ./bin/generate-samples.sh bin/configs/java*.
    IMPORTANT: Do NOT purge/delete any folders/files (e.g. tests) when regenerating the samples as manually written tests may be removed.
  • File the PR against the correct branch: master (upcoming 7.x.0 minor release - breaking changes with fallbacks), 8.0.x (breaking changes without fallbacks) (I think I've choosen correctly?)
  • If your PR is targeting a particular programming language, @mention the technical committee members, so they are more likely to review the pull request.

Technical Committee

@TiFu (2017/07) @taxpon (2017/07) @sebastianhaas (2017/07) @kenisteward (2017/07) @Vrolijkx (2017/09) @macjohnny (2018/01) @topce (2018/10) @akehir (2019/07) @petejohansonxo (2019/11) @amakhrov (2020/02) @davidgamero (2022/03) @mkusaka (2022/04) @joscha (2024/10)

Related issues

Fix #21132
Fix #11307

@tikki100 tikki100 force-pushed the typescript-fetch-fix-null-values branch from 84141d1 to d095ad9 Compare April 23, 2025 13:54
@tikki100 tikki100 marked this pull request as ready for review April 23, 2025 14:11
@tikki100 tikki100 force-pushed the typescript-fetch-fix-null-values branch from 95c8aa6 to f0aef4a Compare April 23, 2025 14:13
@macjohnny
Copy link
Member

thanks for the fix!

@macjohnny
Copy link
Member

@tikki100 please update the samples

@tikki100
Copy link
Author

tikki100 commented Apr 23, 2025

@macjohnny I've re-ran on all typescript-fetch samples :) It changed a package.json for some reason as well, so I've added that as a seperate commit, so it can be git reset head^1 if the CI/CD fails again - idk if it's because I'm using the ./run-in-docker.sh script.

Thanks for the swift response! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants