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

FIO-8798: updated shape of the data of day component with hidden fields #5760

Merged
merged 8 commits into from
Aug 26, 2024

Conversation

Maria-Golomb
Copy link
Contributor

Link to Jira Ticket

https://formio.atlassian.net/browse/FIO-8798

Description

The formation of the data string for the Day component with hidden fields was changed.
if a element of the day component is hidden, it should be excluded from the string instead of populating with ‘00’

Examples (new years): 

day / year == ‘01/2024’

month / year = ‘01/2024’

day / month = ‘01/01’

Why have you chosen this solution?

Use this section to justify your choices

Breaking Changes / Backwards Compatibility

Use this section to describe any potentially breaking changes this PR introduces or any effects this PR might have on backwards compatibility

Dependencies

formio/core#140

How has this PR been tested?

autotests

Checklist:

  • I have completed the above PR template
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • My changes include tests that prove my fix is effective (or that my feature works as intended)
  • New and existing unit/integration tests pass locally with my changes
  • Any dependent changes have corresponding PRs that are listed above

values[YEAR] === '0000' ||
values[DAY] === '' ||
values[MONTH] === '' ||
values[YEAR] === '');
}

getValidationFormat() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this method still work when some part of date is hidden?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@TanyaGashtold TanyaGashtold merged commit 8eb4f28 into master Aug 26, 2024
5 checks passed
lane-formio pushed a commit that referenced this pull request Aug 28, 2024
…ds (#5760)

* FIO-8798: updated shape of the data of day component with hidden fields

* FIO-8798: add conditionOperatorsSettings for Day component

* FIO-8798: update core version

* FIO-8798: add yarn.lock

* Revert "FIO-8798: add yarn.lock"

This reverts commit 80c5127.

* Revert "FIO-8798: update core version"

This reverts commit 12a30c3.

* FIO-8798: update validation format

* FIO-8798: update partial day check
lane-formio pushed a commit that referenced this pull request Sep 5, 2024
…ds (#5760)

* FIO-8798: updated shape of the data of day component with hidden fields

* FIO-8798: add conditionOperatorsSettings for Day component

* FIO-8798: update core version

* FIO-8798: add yarn.lock

* Revert "FIO-8798: add yarn.lock"

This reverts commit 80c5127.

* Revert "FIO-8798: update core version"

This reverts commit 12a30c3.

* FIO-8798: update validation format

* FIO-8798: update partial day check
lane-formio added a commit that referenced this pull request Sep 12, 2024
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.

2 participants