fix: Parse API response correctly in admin meetings page#790
Merged
Conversation
The admin meetings page was treating API responses as raw arrays instead of
extracting data from the response object structure. The APIs return
{ meetings: [...] }, { series: [...] }, and { meeting, attendees } but
the frontend was assigning the entire response object directly.
- Fix loadMeetings() to extract data.meetings from response
- Fix loadSeries() to extract data.series and handle missing group filter
- Fix viewMeeting() to properly merge meeting and attendees from response
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Allows admins to schedule a meeting directly from the committee list without having to navigate to the meetings page first. - Added Schedule button on each committee row - Auto-opens scheduling modal via URL parameter (?action=schedule) - Working group is pre-selected in the modal Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Prevents the modal from re-opening if the user refreshes the page or bookmarks it while the modal is open. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
bokelley
added a commit
that referenced
this pull request
Jan 16, 2026
* fix: Parse API response correctly in admin meetings page
The admin meetings page was treating API responses as raw arrays instead of
extracting data from the response object structure. The APIs return
{ meetings: [...] }, { series: [...] }, and { meeting, attendees } but
the frontend was assigning the entire response object directly.
- Fix loadMeetings() to extract data.meetings from response
- Fix loadSeries() to extract data.series and handle missing group filter
- Fix viewMeeting() to properly merge meeting and attendees from response
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* feat: Add Schedule button to admin working groups page
Allows admins to schedule a meeting directly from the committee list
without having to navigate to the meetings page first.
- Added Schedule button on each committee row
- Auto-opens scheduling modal via URL parameter (?action=schedule)
- Working group is pre-selected in the modal
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* chore: Clear action param from URL after opening schedule modal
Prevents the modal from re-opening if the user refreshes the page
or bookmarks it while the modal is open.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
bokelley
added a commit
that referenced
this pull request
Jan 20, 2026
* fix: Parse API response correctly in admin meetings page
The admin meetings page was treating API responses as raw arrays instead of
extracting data from the response object structure. The APIs return
{ meetings: [...] }, { series: [...] }, and { meeting, attendees } but
the frontend was assigning the entire response object directly.
- Fix loadMeetings() to extract data.meetings from response
- Fix loadSeries() to extract data.series and handle missing group filter
- Fix viewMeeting() to properly merge meeting and attendees from response
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* feat: Add Schedule button to admin working groups page
Allows admins to schedule a meeting directly from the committee list
without having to navigate to the meetings page first.
- Added Schedule button on each committee row
- Auto-opens scheduling modal via URL parameter (?action=schedule)
- Working group is pre-selected in the modal
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
* chore: Clear action param from URL after opening schedule modal
Prevents the modal from re-opening if the user refreshes the page
or bookmarks it while the modal is open.
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.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
1 participant
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.
Summary
{ meetings: [...] },{ series: [...] }, and{ meeting, attendees }but the frontend was assigning the entire response object directlyChanges
loadMeetings(): Extractdata.meetingsfrom response instead of using response directlyloadSeries(): Extractdata.seriesand handle case when no working group is selected (series endpoint requires it)viewMeeting(): Properly extractmeetingandattendeesfrom responseTest plan
🤖 Generated with Claude Code