Skip to content

Conversation

@devindford
Copy link
Contributor

Description

Updates the URLs for dbm and auth to only include the port if the env var is defined

How to test

  • Pull this branch
  • run docker-compose --profile dbm up
  • goto localhost:3000
  • ensure the product order ticker shows up
  • run docker-compose --profile dbm down
  • goto your .env.local file and remove the NEXT_PUBLIC_DBM_PORT
  • run docker-compose --profile dbm up
  • check the network tab for the site now and you should see that the get-item route is failing and the url should show without the port now

@devindford devindford merged commit f03d9b2 into main Apr 13, 2023
@devindford devindford deleted the devin.ford/update-dbm+auth-url branch April 13, 2023 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants