Skip to content

Conversation

Keatsm
Copy link
Contributor

@Keatsm Keatsm commented Jun 13, 2025

Users can now upload a pdf of their academic statement which will load courses into Circles. This option is in the menu where you can upload exported planners.

Further testing is needed on different academic statements as there is no formal specification for what an academic statement consits of.

Users can now upload a pdf of their academic statement which will load
courses into Circles. This option is in the menu where you can
upload exported planners.

Further testing is needed on different academic statements as there
is no formal specification for what an academic statement consits
of.
@Keatsm Keatsm requested a review from a team as a code owner June 13, 2025 08:24
@Keatsm Keatsm marked this pull request as draft June 13, 2025 08:24
Copy link

Test Results

90 tests   89 ✅  50s ⏱️
 3 suites   1 💤
 2 files     0 ❌

Results for commit 71a9716.

@lhvy
Copy link
Collaborator

lhvy commented Jun 29, 2025

Merging this before 331-staging is going to be pretty painful due to the massive rewrites of user data. I'd really recommend either rebasing this onto 331-staging, or waiting for 331-staging to merge and rebasing off of dev.

Also my academic statement will very likely trigger a few edge conditions if you'd like to test with mine - follow up on Discord.

@matthew-lim-matthew-lim
Copy link
Collaborator

Should rebase since 331 got merged in. And pass unit tests.

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.

3 participants