Skip to content

Conversation

AdamVyborny
Copy link
Contributor

Přidávám ještě check na ten data type support do configu a ty chybějící properties do manifestu

@AdamVyborny AdamVyborny requested a review from ondrajodas June 17, 2024 10:44
ondrajodas
ondrajodas previously approved these changes Jun 18, 2024
@ondrajodas ondrajodas self-requested a review June 18, 2024 07:37
@ondrajodas ondrajodas dismissed their stale review June 18, 2024 07:39

viz comment

@AdamVyborny AdamVyborny requested a review from ondrajodas June 18, 2024 10:06
@AdamVyborny AdamVyborny marked this pull request as draft June 18, 2024 11:46
@AdamVyborny
Copy link
Contributor Author

@AdamVyborny AdamVyborny marked this pull request as ready for review June 18, 2024 11:51
@AdamVyborny
Copy link
Contributor Author

Ještě tu bude potřeba vyřešit asi tohle: https://keboolaglobal.slack.com/archives/C06LY0QJ3C5/p1718710755714529?thread_ts=1718699241.839479&cid=C06LY0QJ3C5

Take nakonec necháme být. Tak můžeš CR @ondrajodas pls

enum DatatypeSupport: string
{
case AUTHORITATIVE = 'authoritative';
case HINT = 'hint';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

je hints ne hint

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point

@AdamVyborny AdamVyborny force-pushed the adamvyborny-PST-1376 branch 2 times, most recently from d670414 to e0e9c58 Compare June 18, 2024 13:56
@AdamVyborny AdamVyborny force-pushed the adamvyborny-PST-1376 branch from e0e9c58 to 4d20441 Compare June 18, 2024 13:58
@AdamVyborny AdamVyborny requested a review from ondrajodas June 18, 2024 14:00
@AdamVyborny AdamVyborny merged commit bccd4e3 into master Jun 18, 2024
@AdamVyborny AdamVyborny deleted the adamvyborny-PST-1376 branch June 18, 2024 14:25
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.

3 participants