Skip to content

fix: do not rerender nAPM confirmation page when polling [POS-1633]#262

Merged
szymon-mysiak-cko merged 5 commits into
masterfrom
fix-polling
May 21, 2026
Merged

fix: do not rerender nAPM confirmation page when polling [POS-1633]#262
szymon-mysiak-cko merged 5 commits into
masterfrom
fix-polling

Conversation

@szymon-mysiak-cko
Copy link
Copy Markdown
Contributor

@szymon-mysiak-cko szymon-mysiak-cko commented May 19, 2026

Description

When doing polling, in most cases returned responses are empty with pending state. If the initial response contained some data e.g. qr code and consequent polling responses were empty, the qr code disappeared after rerender. This PR sets the behavior that render for pending state happens only once.

Solution

Demo

Checklist

  • I bumped the version of the project using yarn bump-version
  • I have checked the code for any potential issues
  • I tested my changes in the browser

Notes

Jira Issue

@szymon-mysiak-cko szymon-mysiak-cko marked this pull request as ready for review May 20, 2026 15:51
Copilot AI review requested due to automatic review settings May 20, 2026 15:51
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes an issue where repeated background polling responses in the PENDING state caused the APM Pending view to re-render unnecessarily, which could reset the pending UI state (e.g., countdown/instructions).

Changes:

  • Track the previous payment state before applying the newly returned config.state.
  • Avoid re-rendering APMViewPending when the state remains PENDING across polling cycles (PENDING → PENDING).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@szymon-mysiak-cko szymon-mysiak-cko changed the title Fix polling fix: do not rerender nAPM confirmation page when polling [POS-1633] May 20, 2026
@szymon-mysiak-cko szymon-mysiak-cko merged commit 59641b5 into master May 21, 2026
5 checks passed
@szymon-mysiak-cko szymon-mysiak-cko deleted the fix-polling branch May 21, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants