Skip to content

DO NOT MERGE chore(migrations): remove test user account#136

Draft
mirdono wants to merge 1 commit into
developmentfrom
chore/remove-test-user-account
Draft

DO NOT MERGE chore(migrations): remove test user account#136
mirdono wants to merge 1 commit into
developmentfrom
chore/remove-test-user-account

Conversation

@mirdono

@mirdono mirdono commented Jun 15, 2026

Copy link
Copy Markdown
Member

Warning

Do not merge yet. Given people some time to create their own accounts on the necessary environments, cf. #135

Remove the default test user from app instances where it was previously
inserted. This disables this account for pipeline dashboard.

How to test

  1. Switch to this PR's branch
  2. Restart the migrations service and verify that the added migration executes correctly.
  3. Try to login into the dashboard using the test account, this should fail now.

Related tickets

  • LBRON-1589
  • LBRON-1641

@mirdono mirdono self-assigned this Jun 15, 2026
@mirdono mirdono marked this pull request as draft June 15, 2026 11:34

@mirdono mirdono left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Marked as draft. To be merged some time after #135 so people have some time to create their own accounts.

@mirdono mirdono force-pushed the fix/harvester-dashboard-credentials branch from b421625 to 9f48375 Compare June 18, 2026 07:40
@mirdono mirdono changed the title chore(migrations): remove test user account DO NOT MERGE chore(migrations): remove test user account Jun 18, 2026
Remove the default `test` user from app instances where it was previously
inserted.  This disables this account for pipeline dashboard.
@mirdono mirdono force-pushed the chore/remove-test-user-account branch from c40bd7b to 58d0888 Compare June 18, 2026 08:01
@mirdono mirdono deleted the branch development June 18, 2026 08:02
@mirdono mirdono closed this Jun 18, 2026
@mirdono

mirdono commented Jun 18, 2026

Copy link
Copy Markdown
Member Author

Github automatically closed this PR after initial target branch was deleted. Reopening it.

@mirdono mirdono reopened this Jun 18, 2026
@mirdono mirdono changed the base branch from fix/harvester-dashboard-credentials to development June 18, 2026 08:08
@mirdono mirdono requested a review from Rahien June 18, 2026 08:09
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.

2 participants