Skip to content

fix: Current Password field not showing if basic auth is enabled#2343

Merged
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:fix-authentication-settings
Jun 16, 2025
Merged

fix: Current Password field not showing if basic auth is enabled#2343
nicomiguelino merged 1 commit intoScreenly:masterfrom
nicomiguelino:fix-authentication-settings

Conversation

@nicomiguelino
Copy link
Contributor

Issues Fixed

In the Settings page:

  • The Current Password field doesn't show up when the Settings page loads, given that basic authentication is enabled.

Description

  • Updated the condition(s) for showing the Current Password field

Checklist

  • I have performed a self-review of my own code.
  • New and existing unit tests pass locally and on CI with my changes.
  • I have done an end-to-end test for Raspberry Pi devices.
  • I have tested my changes for x86 devices.
  • I added a documentation for the changes I have made (when necessary).

@github-project-automation github-project-automation bot moved this to In progress in Anthias Jun 16, 2025
@sonarqubecloud
Copy link

@nicomiguelino nicomiguelino marked this pull request as ready for review June 16, 2025 18:23
@nicomiguelino nicomiguelino requested a review from a team as a code owner June 16, 2025 18:23
@nicomiguelino nicomiguelino merged commit 2323ec4 into Screenly:master Jun 16, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Done in Anthias Jun 16, 2025
@nicomiguelino nicomiguelino deleted the fix-authentication-settings branch June 16, 2025 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant