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 settings password handling #5988

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

absidue
Copy link
Member

@absidue absidue commented Oct 28, 2024

Fix settings password handling

Pull Request Type

  • Bugfix

Description

This pull request fixes the error in the console when opening the settings while a settings password is set. This doesn't cause any usability issues as the code that errors is the code that sets which section is selected in the side bar and we run it on every scroll event, so it will not be noticable unless you have the devtools open and see the error.

Screenshots

error

Testing

  1. Set a settings password
  2. Navigate to a different page e.g. about
  3. Navigate back to the settings
  4. You should see no error and after entering the password the active section should be highlighted in the side bar immediately

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 8b3c394

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) October 28, 2024 13:41
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 28, 2024
@FreeTubeBot FreeTubeBot merged commit 17122f9 into FreeTubeApp:development Oct 29, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Oct 29, 2024
PikachuEXE added a commit to Zeabyte/FreeTube-0 that referenced this pull request Oct 29, 2024
* development: (225 commits)
  Invidious API: Add support for auto-dubbed audio tracks (FreeTubeApp#5943)
  Fix settings password handling (FreeTubeApp#5988)
  Translated using Weblate (Serbian)
  Translated using Weblate (Breton)
  Translated using Weblate (Toki Pona)
  Translated using Weblate (Breton)
  Translated using Weblate (Toki Pona)
  Translated using Weblate (Icelandic)
  Bump youtubei.js from 10.5.0 to 11.0.0 (FreeTubeApp#5996)
  Bump shaka-player from 4.11.10 to 4.11.11 (FreeTubeApp#5995)
  Bump eslint-plugin-vue from 9.29.1 to 9.30.0 in the eslint group (FreeTubeApp#5994)
  Translated using Weblate (Latvian)
  Translated using Weblate (Toki Pona)
  Translated using Weblate (Icelandic)
  Translated using Weblate (Hungarian)
  Translated using Weblate (Toki Pona)
  Translated using Weblate (Icelandic)
  Translated using Weblate (Hungarian)
  Migrate FtListChannel and FtListHashtag to the composition API (FreeTubeApp#5945)
  Fix issue FreeTubeApp#5969 (FreeTubeApp#5970)
  ...
@absidue absidue deleted the settings-password branch October 29, 2024 06:42
Soham456 pushed a commit to Soham456/FreeTube that referenced this pull request Dec 5, 2024
SuperAKWA pushed a commit to SuperAKWA/FreeTube that referenced this pull request Jan 24, 2025
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.

5 participants