Closed lucaantonelli closed 1 year ago
Great job! I tried the app a little bit and noticed a few things:
I noticed that color
is an integers used as the index for a list of default colors. Wouldn't it be better to save the HEX code instead and convert it directly to the Color
class? With the current solution, if we were to edit the list of colors, we'd have to change all indexes across the database.
Super cool! I'm merging this. As discussed on Discord, we will take care of issues raised by @theperu and @GBergatto on other PRs.
Known Issues: