Skip to content

Conversation

@zeusongit
Copy link
Contributor

@zeusongit zeusongit commented Oct 15, 2025

Purpose

When a user opens a graph with CPython3 nodes they see a toast notification, if the user closes the graph without dismissing the toast notification, it stays, and when the graph is reopened the notification jumps to a different location on the screen

Screenshot 2025-10-15 152444

Declarations

Check these if you believe they are true

Release Notes

Python migration toast notification should close when closing a graph

Reviewers

(FILL ME IN) Reviewer 1 (If possible, assign the Reviewer for the PR)

(FILL ME IN, optional) Any additional notes to reviewers or testers.

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

@zeusongit zeusongit requested review from a team and Copilot October 15, 2025 19:25
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

See the ticket for this pull request: https://jira.autodesk.com/browse/DYN-9693

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Fix ensures the CPython migration toast notification is closed when a workspace/graph is closed to prevent it persisting and repositioning unexpectedly on reopen.

  • Adds handler OnCurrentWorkspaceCleared to close the toast.
  • Subscribes/unsubscribes to new CurrentWorkspaceCleared event.

Copy link
Contributor

@benglin benglin left a comment

Choose a reason for hiding this comment

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

Weeelp, given what we have in ReadyParams I think this a great solution, @zeusongit. LGTM! 🚀

@zeusongit zeusongit merged commit 5d8fa35 into DynamoDS:master Oct 17, 2025
27 of 30 checks passed
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.

2 participants