Skip to content

feat: enforcing default value for required nullable fields - #100

Merged
armanddidierjean merged 1 commit into
devfrom
fixing-required-nullable-field
Sep 9, 2026
Merged

feat: enforcing default value for required nullable fields#100
armanddidierjean merged 1 commit into
devfrom
fixing-required-nullable-field

Conversation

@maximeroucher

@maximeroucher maximeroucher commented Sep 9, 2026

Copy link
Copy Markdown

Description

Summary

Titan codegen can not differentiate between updates optionnal vs creates nullable fields. We need updates fields to remain optionnals as one can decide to not pass a None value to prevent the current field from being overidden. We thus modify creates to have optionnals nullable fields instead of only nullable fields.

Classification

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔨 Refactor (non-breaking change that neither fixes a bug nor adds a feature)
  • 🔧 Infra CI/CD (changes to configs of workflows)
  • 💥 BREAKING CHANGE (fix or feature that require a new minimal version of the front-end)
  • 😶‍🌫️ No impact for the end-users

Impact & Scope

  • Core functionality changes
  • Single module changes
  • Multiple modules changes
  • Database migrations required
  • Other: ...

Testing

  • 1. Tested this locally
  • 2. Added/modified tests that pass the CI (or tested in a downstream fork)
  • 3. Tested in a deployed pre-prod
  • 0. Untestable (exceptionally), will be tested in prod directly

Documentation

  • Updated the docs accordingly :
  • " Docstrings
  • # Inline comments
  • No documentation needed

@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@armanddidierjean
armanddidierjean merged commit 3bbd6a8 into dev Sep 9, 2026
6 checks passed
@armanddidierjean
armanddidierjean deleted the fixing-required-nullable-field branch September 9, 2026 14:34
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.

2 participants