Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

some fixes and updates #253

Merged
merged 7 commits into from
Dec 4, 2024
Merged

some fixes and updates #253

merged 7 commits into from
Dec 4, 2024

Conversation

TimoGlastra
Copy link
Member

@TimoGlastra TimoGlastra commented Dec 2, 2024

Three features / fixed in one PR 🙈

  • Added a development mode that shows internal error messages for easier debugging by LSPs commit
  • Fixed an issue where the PIN screen would get stuck in a loading state when an incorrect PIN was entered commit
  • Fixed an issue with redirect based auth flow if the authorization flow left the in-app browser (e.g. when requiring authentication using the native AusweisApp with the eID card) commit

@TimoGlastra TimoGlastra changed the title feat: add development mode for easier LSP debugging some fixes and updates Dec 2, 2024
Signed-off-by: Timo Glastra <[email protected]>
@@ -165,7 +165,7 @@ export const SlideWizard = forwardRef(
color={isError ? 'danger' : 'primary'}
/>
</Stack>
<AnimatedStack pos="relative" style={animatedStyles} jc="space-between" fg={1}>
<AnimatedStack style={animatedStyles} flex={1} jc="space-between">
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this intentional, and does this not break things on other screens?

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah good question. this is of course not only used in this page. But yes it's intentional. If the content was longer than the screen the screen would overflow for the length of the component above. But will check if this doesn't break anything on the other screens.

@TimoGlastra TimoGlastra merged commit 50a1f2f into main Dec 4, 2024
1 check passed
@TimoGlastra TimoGlastra deleted the fix/some-fixes branch December 4, 2024 11:44
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