search

cs136 / seashell

Seashell is an online environment for editing, running, and submitting C programming assignments.
GNU General Public License v3.0
38 stars 19 forks source link

bug fix #732

Closed yuliswe closed 7 years ago

yuliswe commented 7 years ago

You might have to clear your browser's indexedDB after merge.

kpalway commented 7 years ago

Can we also make it so we don't have to clear IndexedDB.. if the schema has changed just make IndexedDB do an update that clears the database.

e45lee commented 7 years ago

You can do .version(2) in dexie for dexie to do the autoUpgrade for you I think.

yuliswe commented 7 years ago

Not sure how to do that :( it requires both .version(1) and .version(2) to be defined.

e45lee commented 7 years ago

Define both, and the upgrade process should do it for you.

yuliswe commented 7 years ago

Oh well. It requires me to write an upgrade function. I lost track of what's changed :(

kpalway commented 7 years ago

You can just make it clear the database, since that was what you suggested in the first place

e45lee commented 7 years ago

Fine -- leave it as is, and we'll deal with it later. Change the database name though (so instead of seashell-e45lee, do instead seashell2-e45lee).