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

[Bug]: dab update --relationship.fields does not support compound keys. #2491

Open
1 task done
JerryNixon opened this issue Dec 13, 2024 · 0 comments
Open
1 task done
Labels
bug Something isn't working triage issues to be triaged

Comments

@JerryNixon
Copy link
Contributor

JerryNixon commented Dec 13, 2024

What happened?

When using dab update with --relationship.fields you cannot pass multiple keys.

Input

dab update Fake --config "c:\Temp\delme-demo-jerry\dab-config-copy.json" --relationship Series_Character --target.entity Series_Character --cardinality one --relationship.fields "SeriesId:SeriesId,CharacterId:CharacterId"

Output

C:\Temp\delme-demo-jerry>dab update Fake --config "c:\Temp\delme-demo-jerry\dab-config-copy.json" --relationship Series_Character --target.entity Series_Character --cardinality one --relationship.fields "SeriesId:SeriesId,CharacterId:CharacterId"
Information: Microsoft.DataApiBuilder 1.2.11
Information: User provided config file: c:\Temp\delme-demo-jerry\dab-config-copy.json
Loading config file from c:\Temp\delme-demo-jerry\dab-config-copy.json.
Error: Please provide the --relationship.fields in the correct format using ':' between source and target fields.
Error: Failed to update the Entity: Fake.
Error: Could not update the entity: Fake.

The Problem (relationship fields)

https://github.com/Azure/data-api-builder/blob/main/src/Cli/ConfigGenerator.cs#L1595-L1607

Compared To (mapping option)

https://github.com/Azure/data-api-builder/blob/main/src/Cli/Utils.cs#L141-L157

Code of Conduct

  • I agree to follow this project's Code of Conduct
@JerryNixon JerryNixon added bug Something isn't working triage issues to be triaged labels Dec 13, 2024
@JerryNixon JerryNixon changed the title [Bug]: [Bug]: dab update --relationship.fields does not support multiple keys. Dec 13, 2024
@JerryNixon JerryNixon changed the title [Bug]: dab update --relationship.fields does not support multiple keys. [Bug]: dab update --relationship.fields does not support compound keys. Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage issues to be triaged
Projects
None yet
Development

No branches or pull requests

1 participant