Skip to content

Conversation

@camielvs
Copy link
Collaborator

@camielvs camielvs commented Sep 30, 2025

Description

Folded into #1004

Replace all existing name->node id conversions with NodeManager. To facilitate this the useNodeManager hook was added.

Also implements a clearer delineation between io name, io id and node id.

All tasks and nodes will now run on the new node manager id system

Related Issue and Pull requests

Progresses Progresses Shopify/oasis-frontend#261

Type of Change

  • Improvement
  • Cleanup/Refactor
  • Breaking change

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Test Instructions

Additional Comments

Copy link
Collaborator Author

camielvs commented Sep 30, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@camielvs camielvs force-pushed the 09-29-migrate_legacy_nodeid_utilities_to_nodemanager branch from 320479e to 7d107bc Compare October 1, 2025 23:16
@camielvs camielvs force-pushed the 09-29-implement_node_manager branch from 8c27ed8 to 46676ff Compare October 1, 2025 23:16
@camielvs camielvs force-pushed the 09-29-migrate_legacy_nodeid_utilities_to_nodemanager branch from 7d107bc to 3321611 Compare October 7, 2025 22:53
@camielvs camielvs force-pushed the 09-29-implement_node_manager branch from 46676ff to 9cab80e Compare October 7, 2025 22:53
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