Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
pgarrison created a comment on a pull request on AllenInstitute/biofile-finder
> Yeah I'm still unsure about "NAS" and if users will know what that is but Vast is now alot clearer. I think leaving out "local NAS" entirely makes sense. I don't think our users really know wh...
BrianWhitneyAI created a comment on a pull request on AllenInstitute/biofile-finder
> Glad to see us using the word 'vast' instead of cache in much of the UI, as I think that is more clear to our users. Yeah I'm still unsure about "NAS" and if users will know what that is but V...
BrianWhitneyAI created a comment on a pull request on AllenInstitute/biofile-finder
> Nit: the expiration/eviction period is currently 182 days, not 180. We can change this on the backend if desired. Could also say '6 months' on the frontend, or maybe it's not an important distinc...
aswallace created a comment on a pull request on AllenInstitute/biofile-finder
> Is it legal to delete an annotation? Or set the value to null? Technically yes, passing an empty array as the values effectively deletes annotations when you use the MMS `put` endpoint. Howeve...
kmitcham created a comment on a pull request on AllenInstitute/biofile-finder
Is it legal to delete an annotation? Or set the value to null?
kmitcham created a review on a pull request on AllenInstitute/biofile-finder
Glad to see us using the word 'vast' instead of cache in much of the UI, as I think that is more clear to our users.
saeliddp created a comment on a pull request on AllenInstitute/biofile-finder
Nit: the expiration/eviction period is currently 182 days, not 180. We can change this on the backend if desired. Could also say '6 months' on the frontend, or maybe it's not an important distincti...
pgarrison pushed 1 commit to feature/368-use-local-file-path-annotation AllenInstitute/biofile-finder
- File Path (Vast) instead of File Path (Local) da16e23
pgarrison pushed 2 commits to feature/368-use-local-file-path-annotation AllenInstitute/biofile-finder
BrianWhitneyAI created a comment on a pull request on AllenInstitute/biofile-finder
## Here are some screenshots!  
aswallace created a review comment on a pull request on AllenInstitute/biofile-finder
fixed in [e885706](https://github.com/AllenInstitute/biofile-finder/pull/372/commits/e885706c8a2a107d4fd473cacbeb720e0c5c5088)
aswallace created a review comment on a pull request on AllenInstitute/biofile-finder
removed in [e885706](https://github.com/AllenInstitute/biofile-finder/pull/372/commits/e885706c8a2a107d4fd473cacbeb720e0c5c5088)
aswallace created a review comment on a pull request on AllenInstitute/biofile-finder
removed in [e885706](https://github.com/AllenInstitute/biofile-finder/pull/372/commits/e885706c8a2a107d4fd473cacbeb720e0c5c5088)
aswallace pushed 1 commit to feature/metadata-editing/create-mms-endpoint-logic AllenInstitute/biofile-finder
- remove logic for getting metadata from MMS e885706
aswallace created a review comment on a pull request on AllenInstitute/biofile-finder
Just to be clear, this is currently working as is
aswallace created a review comment on a pull request on AllenInstitute/biofile-finder
Yes, currently we are only passing the annotations that we want to edit
aswallace created a review comment on a pull request on AllenInstitute/biofile-finder
I'm fairly confident we want `PUT`. `PATCH` does not allow us to replace existing values for annotations. I've been testing on `PUT`, and while it does replace all values for a given annotation, it...
aswallace created a review comment on a pull request on AllenInstitute/biofile-finder
Yes, I temporarily kept both of them in for debugging purposes but at this point can be removed