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

[pages] Add Error, Login Page Expired, and Terms #34

Merged
merged 5 commits into from
Mar 19, 2023
Merged

Conversation

dschemp
Copy link
Contributor

@dschemp dschemp commented Feb 11, 2023

Hi,

I previously forked this project for some customization but don't have time and needs to maintain it. I tried to adapt a couple of pages to this project.
This includes (for now) the error page, the login expired page and the TaC page:

error
login expired
terms

I am a bit unhappy about the login expired page and maybe you have some ideas on how to make it look good. ^^

Thanks in advance!

@dschemp
Copy link
Contributor Author

dschemp commented Feb 11, 2023

Yeah, forget what I just said about the login expired page. This might be better:

login expired

@lukin
Copy link
Owner

lukin commented Feb 12, 2023

Hey @dschemp 👋

Thanks for your pull request. Looks good.

You are right. "Click here" links provide poor UX. I also want to think about how to improve it.

@cre8
Copy link

cre8 commented Mar 9, 2023

Is it possible to merge this? Missing the terms page right now

@lukin lukin changed the title Adding Login expired, error and Terms and Conditions pages [pages] Add Error, Login Page Expired, and Terms Mar 19, 2023
@lukin lukin merged commit b622a6d into lukin:master Mar 19, 2023
lukin added a commit that referenced this pull request Oct 24, 2023
Co-authored-by: Anthony Lukin <anthony@lukin.dev>
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.

None yet

3 participants