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

updated so clear fires change event #395

Merged
merged 1 commit into from
Jan 7, 2022
Merged

updated so clear fires change event #395

merged 1 commit into from
Jan 7, 2022

Conversation

gammonpeter
Copy link
Collaborator

No description provided.

@gammonpeter gammonpeter linked an issue Nov 12, 2021 that may be closed by this pull request
@jamesrwelch
Copy link
Contributor

Now getting:

Schema validation failed with the following errors:
  Data path "/polyfills" must be string.

On this branch... what have I done wrong?

@pjmonks pjmonks self-requested a review January 7, 2022 16:28
@pjmonks
Copy link
Contributor

pjmonks commented Jan 7, 2022

I have tested this and have no issue with the UI now. I did find a related error in mdm-core though which prevents you from filtering searches when setting the "Last Updated - To" date filter - this is recorded in MauroDataMapper/mdm-core#230. I think that is separate though, since changing the filter field does trigger the search again, which is what this PR is meant to fix.

@pjmonks pjmonks merged commit 83f5986 into develop Jan 7, 2022
@pjmonks pjmonks deleted the feature/gh-378 branch January 7, 2022 16:31
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.

Advanced Search: Search by Last Updated date fails
3 participants