search

GSTT-CSC / hazen-web-app

Interactive web-based implementation of hazen
Other
0 stars 0 forks source link

Extend Linux instructions #24

Closed sophie22 closed 2 years ago

sophie22 commented 2 years ago

Add installation instructions for Beekeeper Studio for database visualisation and instructions for RabbitMQ. Move Celery broker and backend to config.py

Further improvement to issue #21

tomaroberts commented 2 years ago

Thanks Sophie – what's the motivation for moving the Celery code to config.py?

sophie22 commented 2 years ago

just so that everything is controlled in the config file. I didn't quite understand why the next line says celery.conf.update(app.config) if there's nothing Celery-related in the config file. But now with directly referencing the config file fields, it's redundant again.

sophie22 commented 2 years ago

Happy to remove the Celery config stuff from this branch/PR and can discuss later.

tomaroberts commented 2 years ago

Happy to remove the Celery config stuff from this branch/PR and can discuss later.

Not necessarily. I'm going to test and merge once I get chance to review PR.