Skip to content

Conversation

@SwedishSubmarine
Copy link
Contributor

Lägger till så att aspningskalender finns direkt under vanliga kalendern på hemsidan. Tar även bort facebooken, eftersom den inte längre är del av kommunikationspolicyn så har styret bett oss att ta bort den.

No longer part of communication policy so removing this here as well,
reason for doing it in this branch is uhh it helps the next part look
prettier ^^
Basically just copied it and swapped which calender it's referring to,
the calendar format is way uglier and I am not sure why but I'm going to
just keep it that way because I'm too lazy to figure out google
calendar. This is the same link that is on dtek.se/aspkalender so...
Accidentally left a shitpost...
@SwedishSubmarine SwedishSubmarine requested review from Multipacker, icicel and rachelambda and removed request for Multipacker February 13, 2025 23:30
Multipacker
Multipacker previously approved these changes Feb 13, 2025
Copy link
Contributor

@Multipacker Multipacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mindre cleanup men ser bra ut annars:)

Comment on lines 382 to 384

#~ msgid "Sektionen på Facebook"
#~ msgstr "The divison on Facebook"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Går att ta bort de här raderna eftersom vi inte har facebook boxen längre.

Comment on lines 380 to 382

#~ msgid "Sektionen på Facebook"
#~ msgstr "Sektionen på Facebook"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

De här också

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tog bort de flera gånger för jag inte förstod hur django systemet fungerade T_T
Bra att du fångade att jag missade :)

Forgot to do this yesterday, the link for recharging your student union
card no longer does what it's supposed to, as chalmers seems to not have
that link anymore and only have the option in the mobile app.
Forgot that it would add this in swedish locale as well, fixed.
Footer was linking to /index.html which is just a 404, makes it link to
the homepage instead.
@SwedishSubmarine
Copy link
Contributor Author

jag älskar att bloata mina pr med helt orelaterade ändringar!

Copy link
Contributor

@Multipacker Multipacker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@SwedishSubmarine SwedishSubmarine merged commit 0c4ba6f into main Feb 17, 2025
1 check 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.

2 participants