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

🐞 alternative auth methods: switch back to smartlink tab fails #222

Closed
5 tasks done
edulix opened this issue Aug 23, 2023 · 0 comments
Closed
5 tasks done

🐞 alternative auth methods: switch back to smartlink tab fails #222

edulix opened this issue Aug 23, 2023 · 0 comments
Assignees
Labels
bug Something isn't working epi-1 EPI-1 project

Comments

@edulix
Copy link
Contributor

edulix commented Aug 23, 2023

Preflight checklist

  • I could not find a solution in the existing issues, docs, nor discussions.

Current Behavior

alternative auth methods: switch back to smartlink tab fails.

Expected Behavior

should switch back from pin-tan (email-password plus custom extra fields) to smartlink

Steps To Reproduce

No response

Implementation details

Now when the link is not a smart-link, then the tab to smart link is still shown but mark as disabled and cannot be clicked:

Image

The tab is a html list element with the class disabled set, so you can also hide any disabled li element in this area with theme_css if you want.

Environment

- OS:
- Web browser:

Tasks

Tasks

Master PRs

  1. bug
  2. bug

Stable PRs

  1. bug

Additional context

No response

@edulix edulix added the bug Something isn't working label Aug 23, 2023
@edulix edulix self-assigned this Aug 23, 2023
@edulix edulix added the epi-1 EPI-1 project label Aug 23, 2023
edulix added a commit to sequentech/common-ui that referenced this issue Sep 4, 2023
edulix added a commit to sequentech/common-ui that referenced this issue Sep 4, 2023
edulix added a commit to sequentech/documentation that referenced this issue Sep 5, 2023
Parent issue: sequentech/meta#222

Adding missing documentation of how to send alt-auth to voters
edulix added a commit to sequentech/common-ui that referenced this issue Sep 5, 2023
edulix added a commit to sequentech/documentation that referenced this issue Sep 5, 2023
Parent issue: sequentech/meta#222

Adding missing documentation of how to send alt-auth to voters
edulix added a commit to sequentech/common-ui that referenced this issue Sep 5, 2023
edulix added a commit to sequentech/common-ui that referenced this issue Sep 5, 2023
edulix added a commit to sequentech/common-ui that referenced this issue Sep 5, 2023
@edulix edulix closed this as completed Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working epi-1 EPI-1 project
Projects
None yet
Development

No branches or pull requests

1 participant