Skip to content

Conversation

@StubberG3
Copy link
Contributor

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)

@StubberG3 StubberG3 requested a review from taichan03 August 25, 2025 00:58
@@ -0,0 +1,42 @@
import { useEffect } from 'react';
Copy link
Collaborator

@sahilds1 sahilds1 Nov 4, 2025

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

Copy link
Collaborator

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

@sahilds1 sahilds1 assigned sahilds1 and StubberG3 and unassigned sahilds1 Nov 12, 2025
@sahilds1 sahilds1 changed the base branch from listOfMed to 309-393-test-combined-prs November 26, 2025 22:33
@sahilds1 sahilds1 merged commit 11e84f5 into CodeForPhilly:309-393-test-combined-prs Dec 1, 2025
@taichan03 taichan03 mentioned this pull request Dec 19, 2025
taichan03 added a commit that referenced this pull request Dec 19, 2025
#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
@sahilds1 sahilds1 linked an issue Jan 9, 2026 that may be closed by this pull request
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.

Create role-based views, version 1 (remove forced login)

2 participants