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

Change banner text and links #633

Merged
merged 2 commits into from
Mar 27, 2018
Merged

Change banner text and links #633

merged 2 commits into from
Mar 27, 2018

Conversation

elarb
Copy link
Member

@elarb elarb commented Mar 27, 2018

Changes the banner text and removes the links in the banner:

banner

@elarb elarb changed the title Change banner text, comment out links Change banner text and links Mar 27, 2018
@elarb elarb requested a review from martijnjanssen March 27, 2018 15:18
@skamoen
Copy link
Member

skamoen commented Mar 27, 2018

Photo's are already available, so let's link to the album at https://flitcie.ch.tudelft.nl/Bestuur-61/Area-FiftyLAN

@elarb
Copy link
Member Author

elarb commented Mar 27, 2018

Anything else we want to link to?

@skamoen
Copy link
Member

skamoen commented Mar 27, 2018

The facebook page for updates next year? Something like "Stay updated" linking to https://www.facebook.com/areafiftylan/

@martijnjanssen
Copy link
Contributor

Yeah, I think those contents will be the best choices for the tabs.

Copy link
Contributor

@martijnjanssen martijnjanssen left a comment

Choose a reason for hiding this comment

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

Perfect!

@elarb elarb merged commit 0fbc70d into master Mar 27, 2018
@elarb elarb deleted the change_banner_text branch March 27, 2018 15:46
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.

3 participants