Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

184 fix prod issues #185

Closed
wants to merge 16 commits into from
Closed

184 fix prod issues #185

wants to merge 16 commits into from

Conversation

joekitsmith
Copy link
Contributor

Multiple fixes to allow deployment to prod:

  • API_URL env set for prod and dev docker compose frontend containers
  • For prod, mount admin users list into API container
  • Sort graph filter results by node name but link id
  • Don't check admin user exists in internal db - need to fix checking of external user id in internal db
  • Spin prod containers down before launching
  • Separate run-prod and run-prod-fresh (if ssl cert to be regenerated) scripts
  • Restore functionality where only skills with categories that are in current skill search are displayed initially (to reduce clutter in graph)

@joekitsmith joekitsmith linked an issue May 17, 2023 that may be closed by this pull request
@joekitsmith joekitsmith deleted the 184-fix-prod-issues branch October 18, 2023 09:31
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.

Fix prod issues
1 participant