Skip to content

Conversation

@MImran2002
Copy link
Contributor

@MImran2002 MImran2002 commented Jan 20, 2026

When the user navigate to Bonner Management they have the option to choose year, and to choose the spreadsheet option. When the spreadsheet is downloaded, the "Sophomore Exchange" has many dates while "All Bonner Meetings" has one. This shouldn't occurred as they are swapped. Because of this there is a bigger problem where there could be a requirement aside form All Bonner meeting that can appear multiple times so it should be dynamically created instead of assigned.

Issue Description

The spreadsheets are currently not displaying the correct dates. Some requirements should have multiple dates, like "all Bonner meetings" requirements, but currently, they only have one.

Fixes #1659

Changes

  • created a dictionary where its keys are the table index and values are the dates.
  • loop through the dictionary and find the table index and dates and insert them into the spreadsheet with the table index as the column and the dates being joined are inserted into the specific cells.

Testing

  • Checkout the branch git checkout requirementSpreadsheet
  • Pull the code when you are in the branch git pull
  • Reset the database database/reset_database.sh test or database/reset_database.sh from-backup
  • Navigate to Admin > Bonner Management > Download spreadsheet
  • before change:
image
  • after change:
image

@MImran2002 MImran2002 requested review from BrianRamsay and ojmakinde and removed request for ojmakinde January 21, 2026 22:00
@github-actions
Copy link

View Code Coverage

@BrianRamsay BrianRamsay merged commit a5cbe17 into development Jan 22, 2026
4 checks passed
@BrianRamsay BrianRamsay deleted the requirementSpreadsheet branch January 22, 2026 17:28
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.

Requirements spreadsheets

3 participants