Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Add an error when a user sets a field that doesn't exist #7

Closed
yevgenypats opened this issue Apr 4, 2021 · 0 comments
Closed

Add an error when a user sets a field that doesn't exist #7

yevgenypats opened this issue Apr 4, 2021 · 0 comments
Assignees
@zagronitay @roneli
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@yevgenypats
Copy link
Member

The SDK should panic or at least return an error when resource.Set('column', value) to a column that is not defined
@roneli roneli added enhancement New feature or request good first issue Good for newcomers labels May 3, 2021
zagronitay added a commit to zagronitay/cq-provider-sdk that referenced this issue May 3, 2021
@zagronitay
Added resource set validation cloudquery#7
7ca9ac8 
Added TableValidation to enforce column/table name longer than 63 char cloudquery#9
@roneli roneli mentioned this issue May 3, 2021
Merged
zagronitay added a commit to zagronitay/cq-provider-sdk that referenced this issue May 4, 2021
@zagronitay
Fixed CR
f8f2df7 
Added resource set validation cloudquery#7

Added TableValidation to enforce column/table name longer than 63 char cloudquery#9
zagronitay added a commit to zagronitay/cq-provider-sdk that referenced this issue May 4, 2021
@zagronitay
Fixed CR
79c0cdc 
Added resource set validation cloudquery#7

Added TableValidation to enforce column/table name longer than 63 char cloudquery#9
roneli pushed a commit that referenced this issue May 4, 2021
@zagronitay
Added resource set validation (#20)
7d8a9a3 
* Added resource set validation #7
* Added TableValidation to enforce column/table name longer than 63 char #9
@roneli roneli closed this as completed May 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@zagronitay zagronitay

@roneli roneli

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@zagronitay @yevgenypats @roneli