langcog / wordbankr

R package for accessing Wordbank
http://langcog.github.io/wordbankr/
30 stars 4 forks source link

server connection modes #2

Closed mikabr closed 8 years ago

mikabr commented 9 years ago

The old data loading script had three server connection modes: local, prod, and dev, with prod confusingly being server that runs shiny, not the one that runs django (yeah, we have two separate copies of the database set up... gross...). Do we want to not include dev server connection in the package? Have it require an actual password?

mcfrank commented 9 years ago

that database thing is gross, I agree... it's an optimization thing, right, so that shiny gets its own local copy.

+1 for dev should require password.

dyurovsky commented 9 years ago

Seconded, definitely don't want people connecting to dev without a password, and we should make prod the default.

Dan

On Thu, Jul 16, 2015 at 10:05 PM Michael Frank notifications@github.com wrote:

that database thing is gross, I agree... it's an optimization thing, right, so that shiny gets its own local copy.

+1 for dev should require password.

— Reply to this email directly or view it on GitHub https://github.com/langcog/wordbankr/issues/2#issuecomment-122176713.

mikabr commented 9 years ago

Is "prod" the right label for it?

mcfrank commented 9 years ago

dev and prod seem reasonable in contrast to one another, but maybe we should just make it a default that you generally don't specify unless you want dev? as a habitual argument it's probably a little baffling.