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

Fixing issue #1717 #1718

Merged
merged 1 commit into from Jan 7, 2024
Merged

Fixing issue #1717 #1718

merged 1 commit into from Jan 7, 2024

Conversation

IRRDC
Copy link
Contributor

@IRRDC IRRDC commented Dec 4, 2023

Previous fix for #1568 was incomplete. New expanded fix for #1717. DefaultValue can be an object too.

Q A
Bug fix? [X]
New feature? [ ]
New sample? [ ]
Related issues? fixes #1717

What's in this Pull Request?

Additional check to see if DefaultValue is an object. No more null comparissons, which should have been undefined comparissons.

Previous fix for pnp#1568 was incomplete. New expanded fix for pnp#1717. DefaultValue can be an object too.
@bva999
Copy link

bva999 commented Dec 13, 2023

Are there any possibility to publish new minor version like 3.16.1 with this fix? ((((

@michaelmaillot michaelmaillot merged commit 40e9ce5 into pnp:master Jan 7, 2024
1 check passed
@michaelmaillot
Copy link
Collaborator

Hi @bva999,

Version 3.16.1 has just been released with this fix 🙂

@bva999
Copy link

bva999 commented Jan 9, 2024

Hi @michaelmaillot , thank you for your notice. Unfortunately, the same bug has this recent version 😢 #1733
Waiting for 3.16.2 🙄

IRRDC added a commit to IRRDC/sp-dev-fx-controls-react that referenced this pull request Jan 11, 2024
Bugfix pnp#1718 for bug pnp#1717 was faulty and gets fixed with this.
@IRRDC IRRDC mentioned this pull request Jan 11, 2024
michaelmaillot added a commit that referenced this pull request Jan 18, 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.

Dynamic Forms with TaxonomyFieldTypeMulti fields don't show
3 participants