Skip to content

chore: properly handle string fields in dart.yml#414

Merged
jonas-jonas merged 1 commit intomasterfrom
jonas-jonas/fixDartDescriptions
Apr 28, 2025
Merged

chore: properly handle string fields in dart.yml#414
jonas-jonas merged 1 commit intomasterfrom
jonas-jonas/fixDartDescriptions

Conversation

@jonas-jonas
Copy link
Member

@jonas-jonas jonas-jonas commented Apr 1, 2025

The keto description contains a : which trips up the parser of the dart generator. The solution is to wrap the string in "", to tell the parser explicitely that this is a string and should be treated as such.

Related Issue or Design Document

Fixes #411

Checklist

  • I have read the contributing guidelines
    and signed the CLA.
  • I have referenced an issue containing the design document if my change
    introduces a new feature.
  • I have read the security policy.
  • I confirm that this pull request does not address a security
    vulnerability. If this pull request addresses a security vulnerability, I
    confirm that I got approval (please contact
    security@ory.sh) from the maintainers to push
    the changes.
  • I have added tests that prove my fix is effective or that my feature
    works.
  • I have added the necessary documentation within the code base (if
    appropriate).

Further comments

@jonas-jonas jonas-jonas self-assigned this Apr 1, 2025
@jonas-jonas jonas-jonas merged commit bc1ebe3 into master Apr 28, 2025
4 checks passed
@jonas-jonas jonas-jonas deleted the jonas-jonas/fixDartDescriptions branch April 28, 2025 10:43
Jorgagu pushed a commit to Jorgagu/ory-sdk that referenced this pull request Jul 23, 2025
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.

Keto Java client not updated

2 participants