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

Ugent logo #252

Merged
merged 3 commits into from
May 23, 2024
Merged

Ugent logo #252

merged 3 commits into from
May 23, 2024

Conversation

masinnae
Copy link
Contributor

fixt (normaalgezien) ugent logo op loginpagina en navbar

@AronBuzogany
Copy link

7503982

@reyniersbram
Copy link
Contributor

Ik heb niet het gevoel dat het probleem opgelost is.
Je kan met npm run build en npm run serve testen hoe het er na deployment uit zal zien. De login zal hier niet werken, maar je kan wel op de login-pagina al zien dat het logo niet ingeladen wordt.

@masinnae
Copy link
Contributor Author

masinnae commented May 23, 2024

Ik heb niet het gevoel dat het probleem opgelost is. Je kan met npm run build en npm run serve testen hoe het er na deployment uit zal zien. De login zal hier niet werken, maar je kan wel op de login-pagina al zien dat het logo niet ingeladen wordt.

Bij mij wordt dit wel ingeladen als ik build en serve. Het zou ook moeten werken met die @ anders zouden de andere images ook niet mogen werken.
image

@reyniersbram
Copy link
Contributor

Aha, nu werkt het bij mij ook, weet niet wat er mis was daarnet

@reyniersbram reyniersbram merged commit 47ae91a into dev May 23, 2024
4 checks passed
@reyniersbram reyniersbram deleted the ugent_logo branch May 23, 2024 14:11
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