Skip to content

Enhance and Simplify App Settings#14

Merged
jbouder merged 3 commits intomainfrom
enhance-settings
Aug 20, 2025
Merged

Enhance and Simplify App Settings#14
jbouder merged 3 commits intomainfrom
enhance-settings

Conversation

@jbouder
Copy link
Contributor

@jbouder jbouder commented Aug 20, 2025

The current setup pushes the use of a .env for managing env variables, which can be confusing and doesn't scale well. In order to simplify this, the config should be updated with default values as well as optional values where possible.

Description

  • Updated app settings with default DATABASE_URL
  • Updated app settings to make OIDC_CONFIG_URL optional
  • Updated Readme to make .env optional and only needed for local overrides
  • Updated Dockerfile to remove explicit copy of .env
  • Updated workflow to remove vars as no longer needed

Related Issue

N/A

Motivation and Context

  • Simplifies the process of getting up and running
  • Better supports CI/CD automation where environment variables are more likely to be set in a pipeline

How Has This Been Tested?

  • Local Testing

Screenshots (if appropriate):

@jbouder jbouder added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 20, 2025
@jbouder jbouder changed the title Enhance/Simplify App Settings Enhance and Simplify App Settings Aug 20, 2025
@jbouder jbouder requested a review from marshyski August 20, 2025 12:35
Copy link
Member

@marshyski marshyski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jbouder jbouder merged commit 5be1ebb into main Aug 20, 2025
7 checks passed
@jbouder jbouder deleted the enhance-settings branch August 20, 2025 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants