Rename 'Click To Enter New Patient' button to 'Enter New Patient'#511
Open
c-tonneslan wants to merge 248 commits into
Open
Rename 'Click To Enter New Patient' button to 'Enter New Patient'#511c-tonneslan wants to merge 248 commits into
c-tonneslan wants to merge 248 commits into
Conversation
…reflect public vs. admin login
…; restyle logout button; fix css of mobile nav links; remove login sidenav
… be public; edits remain admin only
…ancer-main into balancer/309/login
Release: v1.1.0
…ed-deployment refactor(ci): clean up docker and kubernetes deployments
Release: v1.1.1
…ot-public [CodeForPhilly#393] Allow unauthenticated users to use the chatbot
…sh-workaround [CodeForPhilly#382] Comment with workaround for PostgreSQL crash
# Conflicts: # deploy/manifests/balancer/base/kustomization.yaml # deploy/manifests/balancer/overlays/production/kustomization.yaml # deploy/manifests/balancer/overlays/sandbox/kustomization.yaml
# Conflicts: # deploy/manifests/balancer/base/kustomization.yaml # deploy/manifests/balancer/overlays/sandbox/kustomization.yaml
database.yaml already has namespace: cloudnative-pg hardcoded. The kustomize patch was a no-op replacing the same value.
The Database resource must live in the cloudnative-pg namespace alongside the Cluster CRD. Kustomize's namespace field overrides all resources, making it impossible to keep a cross-namespace resource in the app overlay. Moved to infra/cloudnative-pg/balancer-database.yaml in cfp-sandbox-cluster.
Kustomize's namespace field overrides ALL resources, making it impossible to keep the Database in cloudnative-pg namespace within the main overlay. Created a separate kustomization at overlays/sandbox/cnpg/ without the namespace field, so the Database stays in cloudnative-pg namespace. This keeps sandbox and production databases separate - production overlay has no CNPG resources.
…HTTP→HTTPS redirect)
Adds the balancer-db-credentials SealedSecret to the sandbox CNPG overlay so cnpg can set the balancer role password. The sealed secret uses the cluster's sealed-secrets controller cert and targets the cloudnative-pg namespace.
…ndbox-cluster The Database CR and SealedSecret should live in cfp-sandbox-cluster under the cloudnative-pg namespace, not in the app overlay.
[CodeForPhilly#187] [IMPROVE] Documentation
feat: integrate Gateway API, CORS config, and CNPG database for sandbox
…ion-boundaries [auth] fix auth problems permission boundaries
…-email-activation [auth] account registration form and email activation
Refactor CORS_ALLOWED_ORIGINS to remove empty strings.
Refactor CORS_ALLOWED_ORIGINS to remove empty strings.
…en-refresh [auth] Update login page, add token refresh, and fix password reset flow
Closes CodeForPhilly#509. Drops 'Click To' for accessibility (not every user uses a click-capable device) and to match standard concise button labeling. Signed-off-by: Charlie Tonneslan <cst0520@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #509.
Drops the leading 'Click To' on the patient-summary button for accessibility (not every user is on a click-capable device) and to match the standard concise button labeling convention.