Skip to content

feat: update username and password#328

Open
saptarshi-datazip wants to merge 2 commits intostagingfrom
feat/updUser
Open

feat: update username and password#328
saptarshi-datazip wants to merge 2 commits intostagingfrom
feat/updUser

Conversation

@saptarshi-datazip
Copy link
Copy Markdown
Collaborator

Description

Added update username and password feature.

  • created public route for username and password updating feature only works when user is logged in session
  • updated swagger docs
  • created make command to run directly in terminal to update username and password

Fixes update user issue

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Tested by changing the default username and password created on starting the docker compose up.
  • Also created new user using the make command and updated password for that.
  • All users currently have the same view of dashboard and all jobs are visible

Screenshots or Recordings

image image image image

Related PR's (If Any):

@saptarshi-datazip saptarshi-datazip changed the base branch from master to staging February 25, 2026 16:16
type UpdateCredentialsRequest struct {
Username string `json:"username" validate:"required" example:"newadmin"`
Password string `json:"password" validate:"required" example:"newpassword"`
}
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

we can use the old struct also. just kept this for swagger and separation.

@saptarshi-datazip saptarshi-datazip requested review from deepanshupal09-datazip and removed request for vishalm0509 March 5, 2026 12:13
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.

1 participant