mjennings061 / viking-log-keeper

661 VGS - Collate gliding log sheets into a database and visual dashboard.
https://viking.streamlit.com/
MIT License
5 stars 0 forks source link

Use keyring to store credentials. Added Config dataclass #37

Closed mjennings061 closed 6 months ago

mjennings061 commented 6 months ago

Closes #36

mjennings061 commented 6 months ago

@CameronMacG and @dgscwhite I need your assistance reviewing this please. Can you do the following to test this works:

  1. Clone the repo (if you have not already)
git clone https://github.com/mjennings061/viking-log-keeper.git
  1. Navigate into the directory:
cd viking-log-keeper
  1. Checkout this branch
git checkout 36-store-credentials-using-keyring
  1. Install the app (ensure you have python installed). The . is important below
python -m pip install --upgrade .
  1. Run the dashboard
viking-dashboard
  1. Enter in the database credentials from your email (it will auto prompt you)
  2. Go to the dashboard http://localhost:8501
DAbbott99 commented 6 months ago

Running viking-dashboard gives the following error:

PS C:\Users\Daniel\viking-log-keeper> viking-dashboard
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\Daniel\AppData\Local\Programs\Python\Python311\Scripts\viking-dashboard.exe\__main__.py", line 4, in <module>
ModuleNotFoundError: No module named 'dashboard'
mjennings061 commented 6 months ago

Running viking-dashboard gives the following error:

PS C:\Users\Daniel\viking-log-keeper> viking-dashboard
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\Daniel\AppData\Local\Programs\Python\Python311\Scripts\viking-dashboard.exe\__main__.py", line 4, in <module>
ModuleNotFoundError: No module named 'dashboard'

Thank you for reviewing so quickly @DAbbott99. I will have a look today to see if I can debug that. Good to see the update-logs is working better for you

mjennings061 commented 6 months ago

@DAbbott99 Can you pretty please open powershell and run the following?

cd viking-log-keeper
git pull
python -m pip install --upgrade .
viking-dashboard