-
Notifications
You must be signed in to change notification settings - Fork 3
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
Renaming according to vote in Discord #177
Renaming according to vote in Discord #177
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, ty!
Frontend still to be done. So we shouldn’t merge before this😋. |
@TheRealKasumi and @danielbayerlein: lgtm, as I started the PR please review as well. |
Looks good to me so far. I think only the logo is left. So we are almost there. To not block the publishing we might continue with a placeholder. One of the pictures from the discord. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, even if it is not the placeholder logo mentioned above :-), ty for light and dark theme!
I also adjusted the case and removed the logo for now. We can bring it back at some point if we want but it's not really relevant. With that I would go ahead and merge it. I tested everything manually and besides the logo loading issue (see discord) everything works fine. We can fix this as a follow-up. |
closes #174
closes #176
Renaming according to vote in Discord, (see #176)