Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Mobility page for ISBs, MRT info #3712

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Add Mobility page for ISBs, MRT info #3712

wants to merge 14 commits into from

Conversation

kagiura
Copy link

@kagiura kagiura commented Apr 9, 2024 •
edited
Loading

Context: #3675

@vercel Vercel
Copy link

vercel bot commented Apr 9, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nusmods-export ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 17, 2024 5:41pm
nusmods-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 17, 2024 5:41pm

@vercel Vercel
Copy link

vercel bot commented Apr 9, 2024

@kagiura is attempting to deploy a commit to a Personal Account owned by @nusmodifications on Vercel.

@nusmodifications first needs to authorize it.

@kokrui kokrui linked an issue Apr 9, 2024 that may be closed by this pull request
Add Mobility page for ISBs, MRT info #3675
Open
13 tasks
@hewliyang hewliyang mentioned this pull request Jun 13, 2024
Closed
@kagiura @zwliew
feat: ✨ basic implementation of mobility page (for ISBs)
2b0bf12
@kagiura @zwliew
refactor: 🚨 fixed dependency loop + fixed SVCs with gaps during the d…
f1e5b47 
…ay to show "resume" status properly
@kagiura @zwliew
refactor: ♻️ refactoring bus apis / util functions
70374d5
@kagiura @zwliew
refactor: ♻️ separate api errors and actually having no more buses
a679805
@kagiura @zwliew
chore: 🗃️ update schedule
4621d91
@kagiura @zwliew
fix: 👽️ new proxy api base url added
47f1449
@kagiura @zwliew
refactor: ✨ show which services are directly accessible from any sele…
28f5887 
…cted stop
@kagiura @zwliew
fix: 🐛 bug fixes (auto campus switch, term/holiday distinction)
35aebf7
@kagiura @zwliew
refactor: ♻️ refactored + optimized the isb pages
c3aefa6
@kagiura @zwliew
feat: 🍱 added bus icons (no implementation yet)
c5a8fe6
@kagiura @zwliew
fix: 🐛 fix mobile having slight horizontal scrolll
be0b2a8
@kagiura @zwliew
refactor: 💄 only hide footer on desktop (where it doesn't fit)
2df462d
@zwliew
Copy link
Member

zwliew commented Nov 17, 2024 •
edited
Loading

Incredible work with this :) I'll clean this up over the holidays and hopefully merge this change in its entirety by the end of the year! First, I plan to get the straightforward stuff like bus-stop-data and API changes merged.

zwliew referenced this pull request Nov 17, 2024
@zwliew
website: Revive ISB bus timings
a83078c 
Straightforward changes necessary to bring back bus timings in Venues.
Most of the work was done by @kagiura in PR #3172.
I'm mainly bringing in the bus stop data and API changes and gluing it
together to make it work with the original bus timing feature.
This was referenced Nov 18, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Mobility page for ISBs, MRT info
2 participants
@kagiura @zwliew