-
Notifications
You must be signed in to change notification settings - Fork 13
Balancer/309/login #377
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
Balancer/309/login #377
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
| @@ -0,0 +1,42 @@ | |||
| import { useEffect } from 'react'; | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this bug get resolved with your changes here?
Sign Out Feature Does Not Terminate Session Properly #379
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per offline convo with @StubberG3, this bug (#379) does get resolved in this PR
#421 Release: v1.1.4 Improvements • [#309] Remove forced login; make app publicly accessible with admin features protected [#377] @StubberG3 • [#309] Add new logout page; restyle admin dropdown and logout button [#377] @StubberG3 • [#309] Allow viewing and downloading of PDFs to be public; edits remain admin only [#377] @StubberG3 • [#393] Switch between "View PDF" and "Download PDF" buttons based on authentication status @sahilds1 • [#387] Restructure chatbot prompt for clarity and context reinforcement [#419] @sahilds1 • [#416] Fix stuck spinner on protected routes by triggering auth verification [#422] @sahilds1 • [#416] Fix redirect issues for unauthorized users [#420] @sahilds1 • [#417] Simplify and make ProtectedRoute more robust [#417] @sahilds1 • Replace loading text with spinner component @sahilds1 • Save patient history data to sessionStorage instead of localStorage @sahilds1 • Fix clearing sessionStorage on every page load for unauthenticated users @sahilds1 • Add admin-only notice to login page @StubberG3 • Remove auth requirement for Chat in navigation @sahilds1 Technical • [#309] Make riskWithSources endpoint public @StubberG3 • [#309] Disable JWT auth for newly public endpoints @StubberG3 • [#309] Separate public and admin API calls @StubberG3 • [#382] Add workaround for PostgreSQL crash with pgvector v0.6.1 on ARM64 [#391] @sahilds1 • [#407] Refactor Docker and Kubernetes deployments @themightychris • feat(ci): update containers-publish workflow @themightychris • fix(ci): remove DJANGO_ALLOWED_HOSTS from secret template @sahilds1 • chore: Update Kind configuration, refactor database and allowed hosts environment variable handling @sahilds1 • Fix Type mismatch in RootState @sahilds1 • Add comprehensive CLAUDE.md documentation @sahilds1 • Update environment configuration structure @sahilds1
Closes the front-end tasks for #309. This shouldn't be pushed to the test site until the JWT authentication for the back-end is also removed for these endpoints:
get_full_list_med (Medication Suggester, Medication List)
get_med_recommend (Medication Suggester)
feedback (Medication Suggester, Leave Feedback)
/chatgpt/conversations (Ask JJ)
/chatgpt/continue_conversation (Ask JJ)