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

Patch FieldCollectionData #1600

Merged
merged 3 commits into from
Sep 15, 2023
Merged

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