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.

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

added a message #541

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

GhostOf0days
Copy link
Member

@GhostOf0days GhostOf0days commented Jan 23, 2025

Last semester, CMUEats was unreliable for opening and closing statuses. We made changes to hopefully make them more reliable. Please contact us (see the footer) if issues persist.

Copy link

vercel bot commented Jan 23, 2025

@GhostOf0days is attempting to deploy a commit to the ScottyLabs Team on Vercel.

A member of the Team first needs to authorize it.

@cirex-web cirex-web self-requested a review February 2, 2025 01:36
@GhostOf0days
Copy link
Member Author

btw, @cirex-web, we shouldn't merge until we make an actual fix tinstead of temp fix i made for dining api specials issue a week or so ago. i can't do rn bc swamped with tests this week.

@cirex-web
Copy link
Contributor

ok

Copy link

vercel bot commented Feb 2, 2025

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

Name Status Preview Comments Updated (UTC)
cmueats ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 2, 2025 11:01pm

@cirex-web
Copy link
Contributor

I think we should merge this after implementing a headless browser solution for the api

@GhostOf0days
Copy link
Member Author

yeah. that + fixing specials/soups parsing functionality.

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.

2 participants