paleobot / pbot-dev

Codebase and initial design documents for pbot client
MIT License
2 stars 2 forks source link

Adding/Deleting/Editing/Loading Images #114

Closed clairecleveland closed 1 year ago

clairecleveland commented 1 year ago

There are issues with images and this may be the system bogging down with such large data. Here is what I know so far using Collection: Claire's Test and Specimen: Claire's specimen test.

aazaff commented 1 year ago

This sounds serious. We will need to investigate this before the hackathon, but I will not commit to having it DONE before hackathon.

NoisyFlowers commented 1 year ago

First bullet: Is this a description of a problem?

Second bullet: I was able to delete the flower image in Claire's test collection/Claire's test specimen 1. However, I could only do this after editing it to add a Category. The lack of Category was likely due to it being created before this field was made required. This misconfiguration should not recur one we clean out the db.

Third bullet: This was an artifact of https://github.com/paleobot/pbot-client/issues/24.

Fourth bullet: This was address in paleobot/pbot-client@674f8775ffc510531b0e0a44a0ebe4f243b2b49c.

clairecleveland commented 1 year ago

First bullet: This tells you that sometimes deleting images works. Second bullet: This tells you sometimes it doesn't and gave an example. Third bullet: Cool, so does not give error anymore and is not a problem anymore? Fourth bullet: @NoisyFlowers So based on above, should this issue be closed?

NoisyFlowers commented 1 year ago

I can't promise there aren't still issues. Image upload is a complicated little stack. But the one's listed here appear to be addressed.