feat(welcome): Welcome page UI #736
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@tthvo I just added the email credentials, do you mind checking? |
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
tthvo
reviewed
Mar 26, 2024
|
Rebase again and resolve conflicts pls :)) |
|
Note that this will require changes to Helm Chart. Will look into it as soon as this is merged. |
connordoman
reviewed
Mar 26, 2024
There was a problem hiding this comment.
I love the redesign! The gif on the front page is a great idea.
However, I don't have the font Inter installed on my system so I get this rendering error:
We are importing Inter correctly in /layout.tsx so all that needs changing is adding a custom CSS variable name. From the Next.js font docs:
linhnnk
requested review from
tthvo and
connordoman
and removed request for
nganphan123
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Welcome to PrivacyPal! 👋
Fixes: #698
Description of the change:
Change the UI for
/.Also added a Feedback form, when the user enters their email and feedback in the form, AWS SES will send the user's email and their feedback to an admin email.
I want to make an automatic confirmation email back to the user as well, i.e.
We have received your feedback...or something like that, but for that I believe we need a domain to send emails to addresses that are not verified in AWS SES. If not possible, we can skip this.Motivation for the change:
The previous home page is pretty monotonous. Should modify the home page to look like a welcome page to introduce the use of the website briefly.