Skip to content

Commit

Permalink
Merge pull request #1645 from michaelmaillot/fix/dynamicform-taxonomy
Browse files Browse the repository at this point in the history
Fix DynamicForm - Taxonomy field
  • Loading branch information
joelfmrodrigues committed Sep 15, 2023
2 parents ce9d6e6 + be5db12 commit edf6253
Show file tree
Hide file tree
Showing 6 changed files with 109 additions and 53 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
"[Nils Andresen](https://github.com/nils-a)",
"[Nishkalank Bezawada](https://github.com/NishkalankBezawada)",
"[Rico van de Ven](https://github.com/RicoNL)",
"[wuxiaojun514](https://github.com/wuxiaojun514)",
"[wuxiaojun514](https://github.com/wuxiaojun514)"
]
},
{
Expand Down
21 changes: 21 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,26 @@
# Releases

## 3.16.0

### New control(s)

- `ViewPicker`: new control ViewPicker [#1439](https://github.com/pnp/sp-dev-fx-controls-react/issues/1439)

### Enhancements

- `FieldCollectionData`: render on page instead of panel and added combobox and peoplepicker controls [#1588](https://github.com/pnp/sp-dev-fx-controls-react/pull/1588)

### Fixes

- `AccessibleAccordion`: fix typo in documentation [#1634](https://github.com/pnp/sp-dev-fx-controls-react/pull/1634)
- `DynamicForm`: fix issue with MultiChoice field [#1510](https://github.com/pnp/sp-dev-fx-controls-react/issues/1510)
- `Localization`: Update dutch translations [#1635](https://github.com/pnp/sp-dev-fx-controls-react/issues/1635)
- `TaxonomyPicker`: suggested item contains double termset name [#1597](https://github.com/pnp/sp-dev-fx-controls-react/issues/1597)

### Contributors

Special thanks to our contributors (in alphabetical order): [Guido Zambarda](https://github.com/GuidoZam), [Nils Andresen](https://github.com/nils-a), [Nishkalank Bezawada](https://github.com/NishkalankBezawada), [Rico van de Ven](https://github.com/RicoNL), [wuxiaojun514](https://github.com/wuxiaojun514).

## 3.15.0

### New control(s)
Expand Down
21 changes: 21 additions & 0 deletions docs/documentation/docs/about/release-notes.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,26 @@
# Releases

## 3.16.0

### New control(s)

- `ViewPicker`: new control ViewPicker [#1439](https://github.com/pnp/sp-dev-fx-controls-react/issues/1439)

### Enhancements

- `FieldCollectionData`: render on page instead of panel and added combobox and peoplepicker controls [#1588](https://github.com/pnp/sp-dev-fx-controls-react/pull/1588)

### Fixes

- `AccessibleAccordion`: fix typo in documentation [#1634](https://github.com/pnp/sp-dev-fx-controls-react/pull/1634)
- `DynamicForm`: fix issue with MultiChoice field [#1510](https://github.com/pnp/sp-dev-fx-controls-react/issues/1510)
- `Localization`: Update dutch translations [#1635](https://github.com/pnp/sp-dev-fx-controls-react/issues/1635)
- `TaxonomyPicker`: suggested item contains double termset name [#1597](https://github.com/pnp/sp-dev-fx-controls-react/issues/1597)

### Contributors

Special thanks to our contributors (in alphabetical order): [Guido Zambarda](https://github.com/GuidoZam), [Nils Andresen](https://github.com/nils-a), [Nishkalank Bezawada](https://github.com/NishkalankBezawada), [Rico van de Ven](https://github.com/RicoNL), [wuxiaojun514](https://github.com/wuxiaojun514).

## 3.15.0

### New control(s)
Expand Down
2 changes: 1 addition & 1 deletion src/common/telemetry/version.ts
Original file line number Diff line number Diff line change
@@ -1 +1 @@
export const version: string = "3.15.0";
export const version: string = "3.16.0";
6 changes: 4 additions & 2 deletions src/controls/dynamicForm/DynamicForm.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -255,8 +255,8 @@ export class DynamicForm extends React.Component<
} else if (fieldType === "TaxonomyFieldType") {
objects[columnInternalName] = {
__metadata: { type: "SP.Taxonomy.TaxonomyFieldValue" },
Label: value[0].name,
TermGuid: value[0].key,
Label: value[0]?.name ?? "",
TermGuid: value[0]?.key ?? "11111111-1111-1111-1111-111111111111",
WssId: "-1",
};
} else if (fieldType === "TaxonomyFieldTypeMulti") {
Expand Down Expand Up @@ -414,6 +414,7 @@ export class DynamicForm extends React.Component<
// trigger when the user change any value in the form
private onChange = async (
internalName: string,
// eslint-disable-next-line @typescript-eslint/no-explicit-any
newValue: any,
additionalData?: FieldChangeAdditionalData
): Promise<void> => {
Expand Down Expand Up @@ -774,6 +775,7 @@ export class DynamicForm extends React.Component<
listId: string,
contentTypeId: string | undefined,
webUrl?: string
// eslint-disable-next-line @typescript-eslint/no-explicit-any
): Promise<any> => {
// eslint-disable-line @typescript-eslint/no-explicit-any
try {
Expand Down
Loading

0 comments on commit edf6253

Please sign in to comment.