-
Notifications
You must be signed in to change notification settings - Fork 2
Support field type conversion for Multi Choice fields #1927
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
Changes from 5 commits
1209c63
086831a
c3eab13
4ddcf19
dd9bab9
475fa24
c6ddd9f
3dff74e
7263cf5
bcbf4b2
86e4e97
0f70030
34b1a12
204c0b3
b94afdd
5cf39c7
61a6075
19249d2
efc5687
9748ff9
ed898ad
8ee7b7e
4372404
b2a0905
3b41506
243549b
655b77a
6aa41fd
91c5461
897cacf
79af77d
818598a
2e71d62
138a4ce
4cccb78
758e13c
be1accf
af661fb
93d50b1
095c929
a2efc24
1498baf
9e837ec
91fe4aa
532b8a5
aa78e61
5799262
549dce7
5871747
f9253a7
83af9aa
18556d8
9fab570
e91f013
58c57ff
9dcafbc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -2,19 +2,80 @@ import React from 'react'; | |||||||||
| import { render } from '@testing-library/react'; | ||||||||||
|
|
||||||||||
| import { ConfirmDataTypeChangeModal, getDataTypeConfirmDisplayText } from './ConfirmDataTypeChangeModal'; | ||||||||||
| import { DATE_TYPE, DATETIME_TYPE, PROP_DESC_TYPES, TIME_TYPE } from './PropDescType'; | ||||||||||
| import { | ||||||||||
| BOOLEAN_TYPE, | ||||||||||
| DATE_TYPE, | ||||||||||
| DATETIME_TYPE, | ||||||||||
| FILE_TYPE, | ||||||||||
| INTEGER_TYPE, | ||||||||||
| MULTI_CHOICE_TYPE, | ||||||||||
| MULTILINE_TYPE, | ||||||||||
| PROP_DESC_TYPES, | ||||||||||
| TEXT_CHOICE_TYPE, | ||||||||||
| TEXT_TYPE, | ||||||||||
| TIME_TYPE, | ||||||||||
| } from './PropDescType'; | ||||||||||
| import { | ||||||||||
| BOOLEAN_RANGE_URI, | ||||||||||
| DATETIME_RANGE_URI, | ||||||||||
| FILELINK_RANGE_URI, | ||||||||||
| INT_RANGE_URI, | ||||||||||
| MULTI_CHOICE_RANGE_URI, | ||||||||||
| MULTILINE_RANGE_URI, | ||||||||||
| TEXT_CHOICE_CONCEPT_URI, | ||||||||||
| TIME_RANGE_URI, | ||||||||||
| } from './constants'; | ||||||||||
|
|
||||||||||
| describe('ConfirmDataTypeChangeModal', () => { | ||||||||||
| const stringType = { | ||||||||||
| rangeURI: 'http://www.w3.org/2001/XMLSchema#boolean', | ||||||||||
| dataType: TEXT_TYPE, | ||||||||||
| }; | ||||||||||
|
|
||||||||||
|
||||||||||
| const stringType = { | |
| rangeURI: 'http://www.w3.org/2001/XMLSchema#boolean', | |
| dataType: TEXT_TYPE, | |
| }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor but when I look at this in the UI the "Single Select" and "Multiple Select" text is capitalized. Does that matter for this test case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. I don't think it matters in this case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new
version 7.Xentry still uses placeholder values for the version and release date ("7.X", "X January 2026") and aTODOmarker in the bullet text. For clarity and consistency with the surrounding release notes, consider updating this entry to the actual version (e.g., 7.14.0-...) and a concrete release date, and replacing theTODOwith a brief description of the change before publishing.