-
Notifications
You must be signed in to change notification settings - Fork 11
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
Newly generated profiles are broken #276
Comments
@bacharakis @alexdor I attempted to debug it locally but I faced some issues. I logged in with new account and though the login page redirected me to localhost, the user was not logged in and not added in the backend, so I cannot reproduce the broken layout issue. |
Did you delete your user from the admin app? if so you should be getting an error that you need to contact an administrator (check the url, the frontend doesn't display or check any error) |
@alexdor I did delete the user from the admin app and I am getting the error. |
Yeah, that’s because if the user is deleted (which means banned) we have to manually enable him again. Could you try logging in with a different user?
…On 6 Mar 2019, 17.29 +0100, Iro Palampougiouki ***@***.***>, wrote:
@alexdor I did delete the user from the admin app and I am getting the error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I did log in with two new users and cannot reproduce the issue. We had a regression yesterday which had this result on the user page and was fixed with pr #265 and #267. @bacharakis are those print screens from today? |
Even if they are from today, it might be cached version (the service worker
helps a lot with the speed of the website but it creates a n+1 cache issue,
there are a few good points for the service workers here
https://gist.github.com/Rich-Harris/fd6c3c73e6e707e312d7c5d7d0f3b2f9 )
…On Wed, 6 Mar 2019 at 17.54, Iro Palampougiouki ***@***.***> wrote:
I did log in with two new users and cannot reproduce the issue. We had a
regression yesterday which had this result on the user page and was fixed
with pr #265 <#265> and #267
<#267>. @bacharakis
<https://github.com/bacharakis> are those print screens from today?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#276 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AICthwEJ8m0D_NrYOXXbQ7MWGkTgbWLcks5vT_LXgaJpZM4bhDOl>
.
|
I am not sure whether it's backend or front end, but newly created profiles have a broken layout and people can't login with these profiles.
The text was updated successfully, but these errors were encountered: