-
Notifications
You must be signed in to change notification settings - Fork 71
add quote of the day #4059
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
base: master
Are you sure you want to change the base?
add quote of the day #4059
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.
I'm not at the computer right not, can't really review. But approving this is urgent and crucial
josemigallas
left a comment
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.
😄
|
@akostadinov I added some backend: akostadinov#2 Now the quote is only shown when the Also, we can add new quotes to the locale file, and it will randomly take one. At the end I couldn't add the George Takei quote because I think it makes no sense without a link to the original toot. And it's not safe to let the React component eval HTML inline in quotes, IMO. |
Write some backend for the quotes
|
This PR is stale because it has not received activity for more than 30 days. Remove stale label or comment or this will be closed in 15 days. |
|
This PR is stale because it has not received activity for more than 30 days. Remove stale label or comment or this will be closed in 15 days. |
Adding quote of the day to login page. I was not sure which place is better so I added it under the login form as well under Red Hat logo.
Let me know if I should it to any additional places for better visibility.