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

🐛 Fix issue where app wont start #190

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

strifel
Copy link
Member

@strifel strifel commented Sep 14, 2022

Typ

  • Bugfix

Implementation

Fix issue calling function on null, when loggedOut is called before app is started fully.

Checklist

Alle erfüllten Boxen ankreuzen. Diese Liste kann auch nach Erstellung der Pull Request vervollständigt werden. Unter diesen Gesichtspunkten wird die Implementation begutachtet.

  • Ich habe die Tests erweitert um zu zeigen, dass der Bugfix/das Feature funktioniert
  • Der neue Code hält sich an die Coding Standards
  • Im Code befinden sich wichtige Kommentare, falls angemessen

Fix issue calling function on null, when loggedOut
is called before app is started fully.
@strifel strifel requested a review from Phantom061 September 14, 2022 19:21
@strifel strifel self-assigned this Sep 14, 2022
@strifel strifel added the bug Something isn't working label Sep 14, 2022
@Phantom061
Copy link
Member

Die Änderung war doch damit man die App auch unter m. aufrufen kann um MOBILE_SITE zu testen oder? weil das funktioniert bei mir immer noch nicht. Auch app. nicht

@strifel
Copy link
Member Author

strifel commented Sep 16, 2022

Nein. Die Änderung behebt nur einen komischen Edge Case, wenn ein User eingeloggt ist, aber der refresh nicht geklappt hat

@strifel
Copy link
Member Author

strifel commented Oct 24, 2022

@Phantom061 bitte reviewen

Copy link
Member

@Phantom061 Phantom061 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Funktioniert bei mir

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants