Skip to content

feat(account-linking): better error handling#562

Merged
Brutus5000 merged 3 commits intoFAForever:developfrom
Ivan-Shaml:feat/account-linking-better-error-handling
Jan 23, 2025
Merged

feat(account-linking): better error handling#562
Brutus5000 merged 3 commits intoFAForever:developfrom
Ivan-Shaml:feat/account-linking-better-error-handling

Conversation

@Ivan-Shaml
Copy link
Contributor

@Ivan-Shaml Ivan-Shaml commented Jan 8, 2025

Better error handling for Steam and GOG account linking. If error occurs during account linking, now the flash message is being scrolled to, in the viewport, and it gets highlighted

Closes #549

@Ivan-Shaml Ivan-Shaml marked this pull request as draft January 8, 2025 20:16
@Ivan-Shaml Ivan-Shaml marked this pull request as ready for review January 11, 2025 20:49
Brutus5000
Brutus5000 previously approved these changes Jan 23, 2025
Copy link
Member

@Brutus5000 Brutus5000 left a comment

Choose a reason for hiding this comment

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

Looks good, I am to lazy to test this locally.

* better error handling for Steam and GOG account linking. If error occurs during account linking, now the flash message is being scrolled to in the view port, and it gets highlighted
@Brutus5000 Brutus5000 merged commit 573115c into FAForever:develop Jan 23, 2025
3 checks passed
@Ivan-Shaml Ivan-Shaml deleted the feat/account-linking-better-error-handling branch February 1, 2025 18:11
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.

Linking steam account goes wrong

2 participants