-
-
Notifications
You must be signed in to change notification settings - Fork 22
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
feat: Add routes for import/export taxonomy #116
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really a great work.
I propose some improvements, but I approve it so you can merge as soon as you want.
@aadarsh-ram make lint can help you in the parser folder :-) |
@alexgarel I've added export functionality (Download and Github) endpoints in the last 2 commits. Please review the same. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really great work @aadarsh-ram
I made some comment, but you are free to ignore them if you want :-) as it's more perfectionist comments.
For me it's all good @aadarsh-ram |
uri = os.environ.get("NEO4J_URI", "bolt://localhost:7687") | ||
access_token = os.environ.get("GITHUB_PAT", "<Add personal access token here>") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alexgarel Please do let me know how this will be added in production/staging.
@alexgarel I'll be merging this to the "multilabelnew" branch. |
What
Related issue(s)