feat: add docdb utils #57
Conversation
src/aind_data_access_api/utils.py
Outdated
| from pymongo import MongoClient | ||
|
|
||
|
|
||
| def is_prefix_valid(prefix: str) -> bool: |
There was a problem hiding this comment.
If this the only function that is requires aind-data-schema, it might be better to remove it and the aind-data-schema dependency for the time being. At the very least, since this package will probably be imported alongside other packages, it will be problematic if the version of aind-data-schema is pinned to a specific version.
There was a problem hiding this comment.
is_record_location_valid uses this function to check that the location field contains a valid data asset name. Should I also remove that check from is_record_location_valid for the time being?
There was a problem hiding this comment.
yeah, let's do those in a separate PR (if needed). Hopefully, people shouldn't be modifying the location.
There was a problem hiding this comment.
Great, I removed those two functions.
closes #48