Skip to content

IS-9422: updates templates to support same-device and cross-device flows#136

Merged
pmhsfelix merged 4 commits intofeature/main/IS-9422-bankid-send-returnUrl-in-backgroundfrom
feature/main/IS-9422-bankid-send-returnUrl-in-background-2
Apr 22, 2026
Merged

IS-9422: updates templates to support same-device and cross-device flows#136
pmhsfelix merged 4 commits intofeature/main/IS-9422-bankid-send-returnUrl-in-backgroundfrom
feature/main/IS-9422-bankid-send-returnUrl-in-background-2

Conversation

@pmhsfelix
Copy link
Copy Markdown

@pmhsfelix pmhsfelix commented Apr 16, 2026

Updates templates to support same-device and cross-device.
Needed by BE PR-9932.
See IS-9422.

@pmhsfelix pmhsfelix marked this pull request as ready for review April 17, 2026 08:58
launch.view.qr.screen-reader.step4.2.2=Mac: Ctrl+Cmd+F
launch.view.qr.screen-reader.outro=Hålla telefonen i stående läge på ett 40cm avstånd från skärmen när du skannar QR-koden.
launch.view.button.show-qr-code=Visa QR-kod
launch.view.button.open-on-this-device=Öppna BankID på den här enheten
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I don't get why we also need messages without the "button" qualifier in the path:
launch.view.show-qr-code launch.view.open-on-this-device does not look to be used.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

They were being used by HAAPI, however I removed them and changed the HAAPI representation so that it uses the exact same messages as the VM templates.

#message("${_templatePrefix}.view.button.show-qr-code")</a>
</form>
<form name="cancel" action="cancel" class="manuallink display-none" method="get">
#end
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Previously, with the auto-start mode, the cancel button was hidden and then displayed after 5 seconds. Now it will be displayed from the beginning for both modes. Did you remove this behaviour voluntarily?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Before my changes, my understanding is that the cancel button was always there when we have a QR code, which always happened for BankID v6.

logger.debug("Not launching the app since the user previously cancelled in the app");
#else
if (!browserStrategy.neverAutoLaunch) {
logger.debug("Launching the app");
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

maybe remove this?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Will do.

@pmhsfelix pmhsfelix merged commit 8633b10 into feature/main/IS-9422-bankid-send-returnUrl-in-background Apr 22, 2026
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.

4 participants