880768 - Add dual DB feature for old Thimble projects hosted at /p/:id

Status: RESOLVED FIXED

(Webmaker Graveyard :: Thimble, defect)

Description

[Jon Buckley [:jbuck]]

Pomax, JP and I have been discussing the best way to continue hosting projects created in thimble-django. We've decided that the best compromise would be to:

1) Add a route for /p/:id and /p/:id/edit to the server that is enabled by an environment variable
2) Add a second sequelize connection for projects according to the thimble-django schema
3) Import all of the thimble-django data into this second DB

Comment 3

[Pomax]

starting with some stubs on https://github.com/Pomax/thimble.webmaker.org/compare/bug880768

Comment 4

[Pomax]

Attachment: https://github.com/mozilla/thimble.webmaker.org/pull/115

database dump is on google drive; this doesn't have its own database, so:

mysql>CREATE DATABASE LegacyThimble;
mysql>CONNECT LegacyThimble;
mysql>SOURCE unpackedfile.sql;
mysql>RENAME TABLE api_page TO LegacyProjects;

then fire up "node app" (otherwise sequelize will build the table for us, which is not very useful, because it'll be empty)

Comment 5

[David Humphrey (:humph)]

Comment on attachment 761091 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/115

Stealing review from Jon.

Comment 6

[David Humphrey (:humph)]

Comment on attachment 761091 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/115

I made a few minor comments/suggestions in the code.  This looks sane to me.  Let's do it on staging and test it a bunch today.

Comment 7

[Pomax]

Comment on attachment 761091 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/115

fresh pull request, same code, pull comments from #112 worked in. Pretty sure this is the same code, routes etc. resolve correctly, but in the interest of safety this does require at least one pair of eyes to run over it just to be sure.

Comment 8

[David Humphrey (:humph)]

Comment on attachment 761091 [details] [review]
https://github.com/mozilla/thimble.webmaker.org/pull/115

On issue to fix, r+ with that.

Comment 9

[Pomax]

landed as https://github.com/mozilla/thimble.webmaker.org/commit/17842143e732f06f84d16f1fabca50cca28b52c1