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.

Sign up for GitHub

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

Jump to bottom

Feature Request: Add method for updating Vertex property #173

Open
kcmvp opened this issue Apr 26, 2024 · 1 comment
Open

Feature Request: Add method for updating Vertex property #173

kcmvp opened this issue Apr 26, 2024 · 1 comment

Comments

@kcmvp
Copy link

kcmvp commented Apr 26, 2024

Right now Vertex can be set only when create a Vertex, for my case I need to update a Vertex property later after create a Vertex.
@kcmvp kcmvp changed the title Feature Request: Add method for updating Vertex Feature Request: Add method for updating Vertex property Apr 26, 2024
@davemcphee
Copy link

I'd love this too.

mdgreenfield added a commit to mdgreenfield/graph that referenced this issue Jan 14, 2025
@mdgreenfield
Add graph.UpdateVertex func
5aa7072 
This PR is a fix for dominikbraun#173

With this PR a Vertex can be replaced allowing callers to first retrieve
an existing Vertex, make all updates, and then call the new function
`graph.UpdateVertex` with the old hash and the new vertex.
@mdgreenfield mdgreenfield mentioned this issue Jan 14, 2025
Open
mdgreenfield added a commit to mdgreenfield/graph that referenced this issue Jan 16, 2025
@mdgreenfield
Add graph.UpdateVertex func
1d2e084 
This PR is a fix for dominikbraun#173

With this PR a Vertex can be replaced allowing callers to first retrieve
an existing Vertex, make all updates, and then call the new function
`graph.UpdateVertex` with the old hash and the new vertex.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@davemcphee @kcmvp