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

Bug: Error for abbreviations for new buildings #477

Open
aomi opened this issue May 30, 2023 · 1 comment · May be fixed by #497
Open

Bug: Error for abbreviations for new buildings #477

aomi opened this issue May 30, 2023 · 1 comment · May be fixed by #497
Assignees
@JoshoTheMosho
Labels
bug Something isn't working frontend Frontend (UI) related issues

Comments

@aomi
Copy link
Member

aomi commented May 30, 2023

Describe the Bug/Issue
The sidebar in the calendar view can't show abbreviations for new buildings.

To Reproduce

  1. Add CSC 225 for Fall 2023.
  2. Notice the abbreviation issue on the sidebar.

Expected Behaviour

The proper abbreviation is shown and not undefined or fallback to non-abbreviated.

Screenshots

image

Desktop/Mobile

  • OS: N/A
  • Browser: N/A
  • Version: N/A
@aomi aomi added bug Something isn't working frontend Frontend (UI) related issues labels May 30, 2023
@szeckirjr
Copy link
Contributor

future ref:

courseup/functions/src/constants.ts

Line 35 in e9bbd61

const buildings: { [key: string]: string } = {

@JoshoTheMosho JoshoTheMosho self-assigned this Jul 27, 2023
@JoshoTheMosho JoshoTheMosho linked a pull request Jul 27, 2023 that will close this issue
Open
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JoshoTheMosho JoshoTheMosho

Labels
bug Something isn't working frontend Frontend (UI) related issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

New building abbreviations support VikeLabs/courseup
3 participants
@aomi @JoshoTheMosho @szeckirjr