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

Replace help_url with that of SchildiChat #27

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

Conversation

luixxiul
Copy link

@luixxiul luixxiul commented Mar 4, 2024

Closes SchildiChat/schildichat-desktop#175

This PR intends to replace help_url with that of SchildiChat.

Signed-off-by: Suguru Hirahara [email protected]


  • I agree to release my changes under this project's license

@luixxiul luixxiul marked this pull request as ready for review March 4, 2024 13:04
SpiritCroc pushed a commit that referenced this pull request Oct 6, 2024
* tokens.ts: improve documentation

Improve variable naming and documentation on the methods in `tokens.ts`.

* rename restoreFromLocalStorage

Since the session data isn't actually stored in localstorage, this feels like a
misleading name.

* Lifecycle: bail out if picklekey is missing

Currently, if we have an accesstoken which is encrypted with a picklekey, but
the picklekey has gone missing, we carry on with no access token at all. This
is sure to blow up in some way or other later on, but in a rather cryptic way.

Instead, let's bail out early.

(This will produce a "can't restore session" error, but we normally see one of
those anyway because we can't initialise the crypto store.)
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.

Link proper FAQ
1 participant