Skip to content

Upstream: Embed PDF in answers, Tooling & Dep upgrades, Bugfixes#80

Merged
yutotakano merged 158 commits intocompsoc-edinburgh:masterfrom
yutotakano:upstream-integration
Mar 12, 2026
Merged

Upstream: Embed PDF in answers, Tooling & Dep upgrades, Bugfixes#80
yutotakano merged 158 commits intocompsoc-edinburgh:masterfrom
yutotakano:upstream-integration

Conversation

@yutotakano
Copy link
Copy Markdown
Member

@yutotakano yutotakano commented Mar 12, 2026

This PR contains upstream changes between 2026/01/25 and 2026/03/12.

The specific range is (51a5804, 2be9850], where 51a5804 was the last commit that was merged into BI File Collection as part of #79.

Exhaustive List of Changes

New Features

Bug fixes

Chores

The following commits are also included, but are not relevant to us

I list them here so that it serves as an exhaustive description of all commits that the PR contains when doing git blame etc in the future.

DB Model Changes

  • answers.AnswerKind was added to distinguish "official" answers vs "personal" answers
  • answers.ExamWord and answers.ExamPageFlow were removed as part of the PyPdfium2 migration
  • The scoreboard.scoreboard_userscore Postgres view now has a column for official answers

Merge

I've done a manual merge with our repo. UTs pass and the frontend looks fine. Thus, if the CI passes, I will merge with full responsibility.

Migration of local database

As part of a bugfix to Docker Compose creating a redundant local database (docker & postgres), if you have previously set up a local database you will need to migrate it. (You'll know if you need to do this, if you run Postgres via D-C and it complains about not finding the postgres user.)

$ docker compose exec -it postgres psql -U docker -d postgres
postgres=# create role temp;
postgres=# grant pg_write_all_data to temp;
postgres=# alter role temp with login;
postgres=# alter role temp with superuser;
postgres=# \q

$ docker compose exec -it postgres psql -U temp -d postgres
postgres=# alter role docker rename to postgres;
postgres=# \q

$ docker compose exec -it postgres psql -U postgres -d postgres
postgres=# drop role temp;
postgres=# alter role postgres with password 'postgres';
postgres=# \q

jaqxues and others added 30 commits January 21, 2026 11:14
Allow markdown in announcement

See merge request vseth/sip-com-apps/community-solutions!591
Make buttons less annoying

See merge request vseth/sip-com-apps/community-solutions!593
Feedback sorting

Closes #383

See merge request vseth/sip-com-apps/community-solutions!594
A lot of migrations seem to have missing dependencies...
melusc and others added 29 commits March 6, 2026 20:50
…-server-error' into 'master'

Fix endpoints "/api/document/{author}/{slug}/files/" and "/api/document/{author}/{slug}/files/{oid}" returning "500 Internal Server Error"

Closes #388 and #387

See merge request vseth/sip-com-apps/community-solutions!611
Add tblib to backend development dependencies to see UT exception tracebacks

See merge request vseth/sip-com-apps/community-solutions!613
Fix auth failing locally when time to expiry < timezone offset to UTC

See merge request vseth/sip-com-apps/community-solutions!612
Django Ninja allows us to retrofit our existing Django application to have
modern type-validated input/output schemas, as well as OpenAPI documentation.

Something similar with FastAPI has proven to be very useful in VISIT for
auto-generating request hooks for the frontend, as well as guaranteeing
safety in HTTP I/O compared to manually checking request.DATA.

OpenAPI documentation is at /api/docs
OpenAPI JSON description is at /api/openapi.json

This commit migrates only the three /api/feedback endpoints.

Unit tests pass. All other routes are unaffected.
Migrate to React Router v6

See merge request vseth/sip-com-apps/community-solutions!579
…e-after-hmr' into 'master'

Migrate to ahooks v2

Closes #367

See merge request vseth/sip-com-apps/community-solutions!615
…source-code' into 'master'

Resolve "Cannot select data in 'Supported Markdown Functions' source code"

Closes #368

See merge request vseth/sip-com-apps/community-solutions!616
…ons' into 'master'

Add mermaid support for exam solutions

Closes #290

See merge request vseth/sip-com-apps/community-solutions!608
add mermaid section to supported functions popup

See merge request vseth/sip-com-apps/community-solutions!617
Use Django Ninja experimentally for feedback APIs

See merge request vseth/sip-com-apps/community-solutions!614
@yutotakano yutotakano merged commit df6e0f7 into compsoc-edinburgh:master Mar 12, 2026
5 checks passed
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.

8 participants