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

Fix issues with shape drawing on initialize #860

Merged
merged 1 commit into from
Jan 9, 2024
Merged

Conversation

andrewkfiedler
Copy link
Contributor

  • Updates the Query model to set the correct cql / filterTree on both itself and the LazyResult object right at construction time, preventing weirdness that could arise from the slight delay we have today
  • Removes a timeout effect that would inadvertantly use old data and possible cause shapes to disappear from the map.
  • Updates the drawing / display component to useCallback

github-actions bot pushed a commit that referenced this pull request Jan 3, 2024
Copy link

github-actions bot commented Jan 3, 2024

An updated dist branch has been created and pushed to origin.
You can use:
"catalog-ui-search": "https://github.com/codice/ddf-ui#catalog-ui-search-dist-860",
in your package.json to use this version in your project.

Remember to use "yarn install --force" if you want to pick up changes each time you make a change to this branch by committing.

 - Updates the Query model to set the right cql / filterTree on both itself and the LazyResult object right at construction time, preventing weirdness that could arise from the slight delay we have today
 - Removes a timeout effect that would inadvertantly use old data and cause the shapes to disappear from the map.
 - Updates the drawing / display component to useCallback so that we can ensure we're handling updates correctly
Copy link

github-actions bot commented Jan 3, 2024

An updated dist branch has been created and pushed to origin.
You can use:
"catalog-ui-search": "https://github.com/codice/ddf-ui#catalog-ui-search-dist-860",
in your package.json to use this version in your project.

Remember to use "yarn install --force" if you want to pick up changes each time you make a change to this branch by committing.

github-actions bot pushed a commit that referenced this pull request Jan 3, 2024
@alexabird
Copy link
Collaborator

Hero Successful

Screen.Recording.2024-01-08.at.5.41.21.PM.mov

@alexabird
Copy link
Collaborator

build now

@cxddfuibot
Copy link
Collaborator

Internal build has been started, your results will be available at build completion.

@cxddfuibot
Copy link
Collaborator

Build SUCCESS See the job results in legacy Jenkins UI or in Blue Ocean UI.

@alexabird alexabird merged commit 8ee7058 into master Jan 9, 2024
2 checks passed
Copy link

github-actions bot commented Jan 9, 2024

You can no longer use:
"catalog-ui-search": "https://github.com/codice/ddf-ui#catalog-ui-search-dist-860",
in your package.json to use this version in your project, as it has been deleted now that the pull request is closed.

Instead, if the pr was merged you can now access these changes by using:
"catalog-ui-search": "https://github.com/codice/ddf-ui#catalog-ui-search-dist-snapshot",

Remember to use "yarn install --force" if you want to pick up changes to this version.

alexabird pushed a commit that referenced this pull request Jan 22, 2024
- Updates the Query model to set the right cql / filterTree on both itself and the LazyResult object right at construction time, preventing weirdness that could arise from the slight delay we have today
 - Removes a timeout effect that would inadvertantly use old data and cause the shapes to disappear from the map.
 - Updates the drawing / display component to useCallback so that we can ensure we're handling updates correctly
alexabird pushed a commit that referenced this pull request Jan 24, 2024
- Updates the Query model to set the right cql / filterTree on both itself and the LazyResult object right at construction time, preventing weirdness that could arise from the slight delay we have today
 - Removes a timeout effect that would inadvertantly use old data and cause the shapes to disappear from the map.
 - Updates the drawing / display component to useCallback so that we can ensure we're handling updates correctly
alexabird pushed a commit that referenced this pull request Jan 24, 2024
- Updates the Query model to set the right cql / filterTree on both itself and the LazyResult object right at construction time, preventing weirdness that could arise from the slight delay we have today
 - Removes a timeout effect that would inadvertantly use old data and cause the shapes to disappear from the map.
 - Updates the drawing / display component to useCallback so that we can ensure we're handling updates correctly
jlcsmith pushed a commit that referenced this pull request Feb 6, 2024
- Updates the Query model to set the right cql / filterTree on both itself and the LazyResult object right at construction time, preventing weirdness that could arise from the slight delay we have today
 - Removes a timeout effect that would inadvertantly use old data and cause the shapes to disappear from the map.
 - Updates the drawing / display component to useCallback so that we can ensure we're handling updates correctly

Co-authored-by: Andrew Fiedler <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants