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

docs: update site demo #6650

Merged
merged 1 commit into from
Dec 19, 2024
Merged

docs: update site demo #6650

merged 1 commit into from
Dec 19, 2024

Conversation

Aarebecca
Copy link
Contributor

No description provided.

Copy link

Walkthrough

This pull request updates the site demo by modifying the port.js file. It removes the idOf import and changes how targetPort is defined for edges in the graph data. Instead of using a function to dynamically assign targetPort, it now explicitly sets targetPort for each edge.

Changes

File Summary
packages/site/examples/element/node/demo/port.js Removed idOf import and changed targetPort assignment from a function to explicit values for each edge.

@@ -1,4 +1,4 @@
import { Graph, idOf } from '@antv/g6';
import { Graph } from '@antv/g6';

Choose a reason for hiding this comment

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

The idOf import has been removed, which means the dynamic assignment of targetPort using idOf is no longer possible. Ensure that the explicit targetPort values (port-1, port-2, port-3) are correct and intended for the demo.

@Aarebecca Aarebecca merged commit e7862dd into v5 Dec 19, 2024
2 checks passed
@Aarebecca Aarebecca deleted the refactor/demo branch December 19, 2024 08:15
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.

1 participant