Skip to content

feat: Add report location backend#1

Open
aalmuhan wants to merge 75 commits into
mainfrom
feature/report-location
Open

feat: Add report location backend#1
aalmuhan wants to merge 75 commits into
mainfrom
feature/report-location

Conversation

@aalmuhan
Copy link
Copy Markdown
Owner

@aalmuhan aalmuhan commented Dec 4, 2025 •
edited by sabualha
Loading

Summary

  • Added /api/report-location endpoint for users to report incorrect dining location data
  • Added /api/reports admin endpoint to view all submitted reports
  • Added location_reports database table to store reports
  • Reports are stored in database (not yet sent to slack because we need the webhook URL)
  • Fixed conceptId lookup to use concept_id_to_internal_id mapping table
  • Added database tests for report location feature

Database Changes

  • Added location_reports table to store user-submitted reports
  • Uses concept_id_to_internal_id table to map frontend conceptId to internal UUID

Test plan

  • Test submitting a report via the frontend
  • Verify reports are stored in database via /api/reports
  • Run pnpm test tests/reportLocation.test.ts

Related PR

  • Frontend PR:

laasyaaki and others added 30 commits August 3, 2025 14:13
Manually added locations
04121c7
cleaned up a bit
259e8d1
latest
612d35f
seperated locations
af470c8
@cirex-web
Merge pull request ScottyLabs#187 from ScottyLabs/flex_only
5992b16 
Manually added off campus flex only locations
@laasyaaki
Revert "Manually added off campus flex only locations"
d15c0fe
@laasyaaki
Merge pull request ScottyLabs#188 from ScottyLabs/revert-187-flex_only
6901972 
Revert "Manually added off campus flex only locations"
@laasyaaki @cirex-web
Revert "Manually added off campus flex only locations"
12cbaf6
@cirex-web
Merge branch 'staging' of https://github.com/ScottyLabs/dining-api in…
fd1d118 
…to staging
@cirex-web
fix: coalesce intervals if gap is 1 min, and do wrap-around coalescing (
b41a6eb 
ScottyLabs#190)

* fix: coalesce intervals if gap is 1 min, and do wrap-around coalescing

* fix: unparseable tokens should be ignored

* chore: remove extraneous comment

* fix: redo the merging algo
@JackHurew
SQL dashboard phase 2 completed-basic ability to overread scrapes wit…
510cd17 
…h our own information.
@JackHurew
Everyone Loves Drizzle
e3437b9
@cirex-web
feat: capital grains override
e9ff386
@cirex-web
chore: remove console.log
1e4c2d0
@cirex-web
fix: fix existing tests
9ad006d
@cirex-web
tests: add tests for overwrites
54fd9b2
@cirex-web
fix: notify when response date header becomes unparseable
56b33b7
@cirex-web
Merge branch 'staging' of https://github.com/ScottyLabs/dining-api in…
ee2dafb 
…to feat/time-overwrites
@cirex-web
Merge branch 'main' into staging
ff507be
@cirex-web
fix: tests
9e27cb5
@cirex-web
refactor: rework the entire parsing algorithm to better accomodate ti…
e2e18ec 
…me slot overrides
@cirex-web
chore: update time slot overwrite description
b1ea9a9
@JackHurew
Dockerfile (Sorry this took so long)
63e016c
@cirex-web
fix: change original Dockerfile to match Railway version, upd readme
b6a710a
@cirex-web
feat: automatic db migrations, move overwrite logic directly under ap…
60ff9b5 
…i endpoint, create bun and pnpm dockerfiles
@cirex-web
chore: finalize migration from bun to pnpm
ebb0ac2
@cirex-web
fix: dependabot
563a80c
@cirex-web
Merge branch 'staging' into SQL_Dashboard
91113db
@cirex-web
fix: env test file
53f1060
@cirex-web
Merge pull request ScottyLabs#192 from ScottyLabs/SQL_Dashboard
18c3256 
SQL dashboard phase 2 completed-basic ability to overread scrapes wit…
cirex-web and others added 18 commits November 13, 2025 10:51
@cirex-web
test: parallel tests
c5d45c9
@cirex-web
test: run all tests
07ad8c4
@cirex-web
test: make tests more readable
f8d6878
@cirex-web
test: add a few more tests
687ed6f
@cirex-web
chore: rename variable
e93638d
@cirex-web
chore: bundle writes into single transaction and add coverage reporting
4ecf5cb
@cirex-web
chore: uncomment reload function
7f6d7ef
@cirex-web
chore: upd readme
079587d
Initial plan
4ac144b
@cirex-web
Refactor timestamps in database.test.ts to use timeToUnixTimestamp fu…
76891b8 
…nction

Co-authored-by: cirex-web <[email protected]>
@cirex-web
chore: upd readme
fdaa26e
@cirex-web
chore: add column names to some columns in the schema
9f7901d
@cirex-web
chore: remove redundant paren
85bafd1
@cirex-web
Merge pull request ScottyLabs#213 from ScottyLabs/copilot/sub-pr-195-…
b9d747c 
…again

Refactor database.test.ts timestamps to use timeToUnixTimestamp helper
@cirex-web
test: add two more tests
8be7e44
@cirex-web
chore: add additional today param check
d9b38ef
@cirex-web
fix: date timezone thing for tests
a40b010
@cirex-web
Merge pull request ScottyLabs#195 from ScottyLabs/feat/time-overwrites
30f44f8 
feat: [breaking] cmueats gets a real database that can store more than 7 days worth of times, and also stores overwrites
@sabualha sabualha force-pushed the feature/report-location branch from d3b6c9d to 5dc9e40 Compare December 8, 2025 19:23
@sabualha sabualha changed the title Feature/report location - Add backend endpoint for reporting incorrect location data feat: Add report location button feature Dec 9, 2025
@sabualha sabualha changed the title feat: Add report location button feature feat: Add report location backend Dec 9, 2025
@cirex-web
Copy link
Copy Markdown

cirex-web commented Dec 23, 2025

hi! if this is ready for review, could you make a PR in the dining api repo?

@sabualha
Copy link
Copy Markdown
Collaborator

sabualha commented Dec 24, 2025

hi! if this is ready for review, could you make a PR in the dining api repo?

hello! yes it is ready for review I'll make it a PR soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@sabualha sabualha

@aalmuhan aalmuhan

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@aalmuhan @cirex-web @sabualha @laasyaaki @JackHurew