search

kylebakerio / mindseal

next-gen cross-platform open-source buzzword-happy SRS
http://mindseal.kylebaker.io
2 stars 1 forks source link

bug: adding shared deck will overwrite existing deck with same name #37

Open kylebakerio opened 8 years ago

kylebakerio commented 8 years ago

plan of action:

  1. how to store these in parallel in model... perhaps we should add one more level of nesting for the shared user than we do for other users. 1a. this will break some other features, but should all be fixable at the server side. Since we return an array of deck objects with their names as properties and their authors as properties, that shouldn't be a problem. (confirm this)
  2. however, because we store the decks in the mindSeal as keys, not as in an array, we will run into problems. We should probably rewrite this, but this breaks a lot of things that use that key... I think (perhaps they would be ok because we can use indexes as keys)? This would also fix the problem we have with turning names into keys being ugly (just dirty adding spaces) and make names of decks much more flexible (allow special characters).

bummer. -_-