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

Save as #1268

Merged
merged 3 commits into from
Sep 18, 2023
Merged

Save as #1268

merged 3 commits into from
Sep 18, 2023

Conversation

shivamka1
Copy link
Collaborator

What changes were proposed in this pull request?

Cabability to save graph with provided graph name.

Why are the changes needed?

Same as above

Does this PR introduce any user-facing change? If yes is this documented?

Yes

How was this patch tested?

Unit tests

Issues

If this resolves any issues, please link to them here, the format is a KEYWORD followed by @_
KEYWORDS available are close, closes, closed, fix, fixes, fixed, resolve, resolves, resolved.
Please delete this text before creating your PR

Are there any further changes required?

No

@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.08% ⚠️

Comparison is base (1784371) 59.34% compared to head (b166636) 59.26%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1268      +/-   ##
==========================================
- Coverage   59.34%   59.26%   -0.08%     
==========================================
  Files         170      170              
  Lines       18149    18172      +23     
==========================================
  Hits        10770    10770              
- Misses       7379     7402      +23     
Files Changed Coverage Δ
raphtory-graphql/src/model/mod.rs 0.00% <0.00%> (ø)

... and 3 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@miratepuffin miratepuffin merged commit 656bc9b into master Sep 18, 2023
@miratepuffin miratepuffin deleted the save-as branch September 18, 2023 12:29
fabianmurariu pushed a commit that referenced this pull request May 21, 2024
* impl save graphs

* save as must save graph with new name
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