Skip to content

Conversation

@mfori
Copy link
Member

@mfori mfori commented Sep 16, 2024

This PR introduce new input_schema properties allowAbsolute and allowRelative used when the editor property is set to datepicker:

@mfori mfori added the t-console Issues with this label are in the ownership of the console team. label Sep 16, 2024
@mfori mfori self-assigned this Sep 16, 2024
@github-actions github-actions bot added this to the 98th sprint - Console team milestone Sep 16, 2024
@mfori mfori changed the title feat(input_schema): datepicker isAbsolute/isRelative properties feat(input_schema): datepicker isAbsolute/isRelative properties and validation Sep 18, 2024
@mfori mfori requested review from B4nan and drobnikj September 25, 2024 13:35
@mfori mfori marked this pull request as ready for review September 25, 2024 13:35
Copy link
Member

@drobnikj drobnikj left a comment

Choose a reason for hiding this comment

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

This package needs to be heavily unit tested and I see no tests here. Please do unit tests for your changes.

Update schema.json to not accept both sets to `false`
Create unit tests
@github-actions github-actions bot added the tested Temporary label used only programatically for some analytics. label Sep 26, 2024
@mfori mfori requested a review from drobnikj September 26, 2024 12:21
Copy link
Member

@drobnikj drobnikj left a comment

Choose a reason for hiding this comment

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

👏 for tests, let's fix last bits and we are good to go.

allowAbsolute: false,
},
},
})).toBe(false);
Copy link
Member

Choose a reason for hiding this comment

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

I would rather test if the error message is correct as the logic is not trivial.

Copy link
Member Author

@mfori mfori Sep 27, 2024

Choose a reason for hiding this comment

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

Added check of error messages

allowRelative: false,
},
},
})).toBe(false);
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

Copy link
Member

@B4nan B4nan left a comment

Choose a reason for hiding this comment

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

only had a brief look

@mfori mfori changed the title feat(input_schema): datepicker isAbsolute/isRelative properties and validation feat(input_schema): datepicker allowAbsolute/allowRelative properties and validation Sep 27, 2024
@mfori mfori requested a review from drobnikj September 27, 2024 09:00
Copy link
Member

@drobnikj drobnikj left a comment

Choose a reason for hiding this comment

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

Nice 👍

@mfori mfori merged commit 843edf7 into master Sep 30, 2024
9 checks passed
@mfori mfori deleted the feat/datepicker-properties branch September 30, 2024 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-console Issues with this label are in the ownership of the console team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants