Skip to content

complete key migration code#647

Merged
parkan merged 7 commits intomainfrom
feat/export-keys
Mar 20, 2026
Merged

complete key migration code#647
parkan merged 7 commits intomainfrom
feat/export-keys

Conversation

@parkan
Copy link
Collaborator

@parkan parkan commented Mar 18, 2026

No description provided.

@parkan parkan force-pushed the feat/export-keys branch from b4b0d72 to 23f5fa8 Compare March 18, 2026 12:18
@parkan parkan force-pushed the feat/export-keys branch from 24680d0 to 751deff Compare March 18, 2026 12:46
@parkan parkan changed the title add export-keys command complete key migration code Mar 18, 2026
@parkan parkan force-pushed the feat/export-keys branch from 2fc67b8 to 72d90f2 Compare March 18, 2026 13:13
@parkan parkan marked this pull request as ready for review March 18, 2026 15:27
@parkan
Copy link
Collaborator Author

parkan commented Mar 19, 2026

@anjor would appreciate a second set of eyes on this since it has some high-risk paths

@parkan parkan requested a review from anjor March 19, 2026 10:41
@parkan parkan force-pushed the feat/export-keys branch 2 times, most recently from 7ccc511 to 46e693f Compare March 20, 2026 09:10
@anjor
Copy link
Collaborator

anjor commented Mar 20, 2026

Awesome, looks good to me.

@parkan
Copy link
Collaborator Author

parkan commented Mar 20, 2026

hmm found a slight wrinkle, hold please

@parkan parkan force-pushed the feat/export-keys branch from 46e693f to 55c29c8 Compare March 20, 2026 11:23
@parkan
Copy link
Collaborator Author

parkan commented Mar 20, 2026

ok, GORM got a little confused so we had to work around it but was able to verify full migration flow manually now

@parkan parkan enabled auto-merge (squash) March 20, 2026 11:25
@parkan parkan merged commit f2688e9 into main Mar 20, 2026
3 checks passed
@parkan parkan deleted the feat/export-keys branch March 20, 2026 11:33
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