search

AY2425S1-CS2103-F13-2 / tp

MIT License
3 stars 5 forks source link

Add support for storing list of associations #96

Open kevin-pek opened 3 days ago

kevin-pek commented 3 days ago

Currently, the application does not support saving the list of associations between vendors and events. We need to extend the storage functionality to store this data in the same JSON format used for other data.

C5hives commented 3 days ago

Given how Event and Vendor objects are currently stored in .json format, I'm not sure if there's a method to store their associations without re-serializing the Event and Vendor objects—which will lead to quite a bit of data duplication in the .json file.

Just something to consider

kevin-pek commented 3 days ago

Would be tricky to implement this. Am thinking of saving it based off the array index of the vendors and events to avoid the duplication issue. Otherwise might be better to introduce a unique id field to the Vendor and Event models.