Skip to content

Conversation

GuidoZam
Copy link
Contributor

Q A
Bug fix? [ ]
New feature? [ x]
New sample? [ ]

What's in this Pull Request?

Added a new implementation for a date field in the FieldCollectionData control.

It also fixes a UI wrong behavior for numeric value that occurs in the FieldCollectionData control when a numeric field is disabled: the input field seems to be editable but it is not.

@joelfmrodrigues
Copy link
Collaborator

joelfmrodrigues commented Sep 12, 2023

@GuidoZam many thanks for the updates. There was another PR to the same control that was created before this one and is now merged, adding two new types of controls: combobox and people picker.
Unfortunately, this resulted in your PR now showing as having conflicts. Could you please get the latest changes from the dev branch and resolve the conflicts?

@GuidoZam
Copy link
Contributor Author

@joelfmrodrigues I'll check as soon as I can and let you know.

@joelfmrodrigues joelfmrodrigues self-assigned this Sep 14, 2023
@joelfmrodrigues joelfmrodrigues added status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. type:enhancement New feature or enhancement of existing capability labels Sep 14, 2023
@joelfmrodrigues
Copy link
Collaborator

@GuidoZam I ended up resolving the conflicts 🙂 no need to bother

@joelfmrodrigues joelfmrodrigues merged commit 2056b09 into pnp:dev Sep 15, 2023
@joelfmrodrigues
Copy link
Collaborator

@GuidoZam many thanks for the new feature and also the fix. This is now merged into the dev branch

@joelfmrodrigues joelfmrodrigues added this to the 3.16.0 milestone Sep 15, 2023
@joelfmrodrigues joelfmrodrigues removed the status:working-on-it Known issue / feature being addressed. Will use other "status:*" labels & comments for more detail. label Sep 15, 2023
@GuidoZam
Copy link
Contributor Author

@joelfmrodrigues many thanks to you!

@GuidoZam GuidoZam deleted the patch-fieldcollectiondata branch September 16, 2023 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement New feature or enhancement of existing capability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants