Merge pull request #39 from alneberg/dotenv_fix

Moved dotenv loading inside config init since it didn't work as it was.
NationalGenomicsInfrastructure · May 6, 2024 · d19b4e7 · d19b4e7
2 parents 4d480be + d2b2737
commit d19b4e7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/daily_read/__main__.py b/daily_read/__main__.py
@@ -11,7 +11,6 @@
 # Installed
 import click
 from dateutil.relativedelta import relativedelta
-import dotenv
 from rich.logging import RichHandler
 
 # Own
@@ -22,7 +21,6 @@
 import daily_read.utils
 
 
-dotenv.load_dotenv()
 config_values = daily_read.config.Config()
 
 rich_handler = RichHandler()

diff --git a/daily_read/config.py b/daily_read/config.py
@@ -1,8 +1,11 @@
 import os
 
+import dotenv
+
 
 class Config(object):
     def __init__(self):
+        dotenv.load_dotenv()
         self.ORDER_PORTAL_URL = os.getenv("DAILY_READ_ORDER_PORTAL_URL")
         self.ORDER_PORTAL_API_KEY = os.getenv("DAILY_READ_ORDER_PORTAL_API_KEY")
         self.REPORTS_LOCATION = os.getenv("DAILY_READ_REPORTS_LOCATION")