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

refetch scenes after new scene on toolbar #10520

Merged
merged 3 commits into from
Jul 8, 2024
Merged

Conversation

aditya-mitra
Copy link
Collaborator

Summary

the newScene and deleteScene functions use the file-browser api to create new scenes. Once the API responds, the scenes would refresh by listening to it.

Subtasks Checklist

Breaking Changes

References

https://tsu.atlassian.net/browse/IR-2947

QA Steps

Copy link
Member

@SYBIOTE SYBIOTE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

basically, refetch has been abstracted away

speigg
speigg previously requested changes Jul 7, 2024
@@ -56,18 +56,18 @@ export default function ScenesPanel() {
getMutableState(EditorState).scenePath.set(scene.key)
}

useRealtime(fileBrowserPath, scenesQuery.refetch)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are already calling useRealtime in FeathersChildReactor for each feathers request. Why do we need to do it again here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's causing a referch for a different query

@HexaField HexaField requested a review from speigg July 7, 2024 23:46
@HexaField HexaField enabled auto-merge July 7, 2024 23:46
@HexaField HexaField added this pull request to the merge queue Jul 7, 2024
Merged via the queue into dev with commit 63d4261 Jul 8, 2024
25 checks passed
@HexaField HexaField deleted the fix/toolbar-newscene branch July 8, 2024 00:07
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.

None yet

4 participants