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

Add Read Support for Status Properties #133

Open
djs-CS opened this issue Apr 17, 2023 · 0 comments
Open

Add Read Support for Status Properties #133

djs-CS opened this issue Apr 17, 2023 · 0 comments

Comments

@djs-CS
Copy link

djs-CS commented Apr 17, 2023

I just opened this PR for adding support for reading status property configs, which was core to my use case.

Even if it's accepted, I'm still unclear on the best way to add support for parsing the status property type into a StatusPropertyConfig while also preventing the StatusPropertyConfig from being used as an input for the DatabaseClient.Create() and DatabaseClient.Update() methods.

Perhaps we could add a methods to DataBaseUpdateRequest and DatabaseCreateRequest that would remove any incompatible PropertyConfigs assigned to them?

I've not investigated this thoroughly, but it doesn't appear to me that the .Update() method has any logic in place to prevent other invalid PropertyConfigs, e.g. formula from being passed in as values.

Would love to hear your thoughts!

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

No branches or pull requests

1 participant