-
Notifications
You must be signed in to change notification settings - Fork 0
Feature kpoland dataset new version #227
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
base: master
Are you sure you want to change the base?
Conversation
|
In this PR:
|
f639183 to
6af27e8
Compare
|
For Monday: Added versioning AND consolidated some javascript functionality (which is why there's so many deletions). Refactoring/updating tests in Jest and Python to match and pushing that shortly.... |
gateway/sds_gateway/api_methods/serializers/dataset_serializers.py
Outdated
Show resolved
Hide resolved
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.
just checking if these "keyword" removals are intentional
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.
no.... I missed where it was being used bc it's in an unexpected place. I'm going to move the KeywordAutocomplete class into a JS file.
08078cb to
0b79b4c
Compare
No description provided.