michaelKurowski / lokim2

An internet messenger that cares about privacy.
4 stars 0 forks source link

Screen glitch while entering chat directly #73

Closed michaelKurowski closed 6 years ago

michaelKurowski commented 6 years ago

List of steps leading to reproduce the error

Expected result I should be shown a chat page, or be redirected if I'm not logged in

Actual result I see glitched page: image

Logs Google Chrome console logs: localhost-1526206842189chat.log

Jordan141 commented 6 years ago

Fixed that bug, leaving a note for myself to include a fix for the bug that came up. Seems that while redirecting to '/' from '/chat', it attempts to reload '/chat' again, fails, returns to '/', resulting in the component for '/' not loading. This is "fixed" by a page refresh. No additional code prompts other than "Cannot redirect to the current route".

Attempted fixes:

Due to the exponentially increasing complexity of this bug, I will attack it another day. For now, the initial issue that this was opened for, has been fixed in https://github.com/michaelKurowski/lokim2/commit/7ad21c733385d51235301d6680d35fc2a53305e1

michaelKurowski commented 6 years ago

@Jordan141 Let's push the PR, for the issue and open another ticket.

Jordan141 commented 6 years ago

Frontend is too brittle in this branch currently to push to master, let me fix more things. @michaelKurowski

Jordan141 commented 6 years ago

With recent commit, page navigation is vastly improved in it's stability. However, it's still not ready to push to master.