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

fix: chunk loading issue - 1 #623

Merged
merged 3 commits into from
Sep 17, 2024
Merged

Conversation

PritishBudhiraja
Copy link
Collaborator

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Chunk loading issue debug phase - 1. Removed lazy from payment element renderer and try to debug and check logs once again.

How did you test it?

Via logs we tested this.

{
  "error": {
    "message": "Loading chunk 872 failed.\n(missing: https://checkout.hyperswitch.io/0.84.14/v0/872.js)",
    "name": "ChunkLoadError",
    "type": "missing",
    "request": "https://checkout.hyperswitch.io/0.84.14/v0/872.js",
    "cause": {
      "name": "React ErrorBoundary ChunkLoadError"
    }
  },
  "componentStack": "\n at Lazy\n at Suspense\n at ErrorBoundary (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:356052)\n at make (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:362824)\n at make (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:378246)\n at div\n at div\n at div\n at RenderPaymentMethods_bs_make (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:589908)\n at Payment_bs_make (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:592213)\n at div\n at LoaderController_bs_make (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:614618)\n at App_bs_make (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:645973)\n at ErrorBoundary (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:356052)\n at make (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:362824)\n at RecoilRoot (https://checkout.hyperswitch.io/0.84.14/v0/app.js:2:534649)\n at div\n at div",
  "eventId": "ac90a5b5f99a4aeb84d8188608bcd21f",
  "componentName": "PaymentElementRendererLazy"
}

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

Copy link

Review changes with SemanticDiff.

@PritishBudhiraja PritishBudhiraja self-assigned this Sep 17, 2024
@PritishBudhiraja PritishBudhiraja added the Ready for Review PR with label Ready for Review should only be reviewed. label Sep 17, 2024
@seekshiva seekshiva merged commit 06e7fab into main Sep 17, 2024
3 checks passed
@seekshiva seekshiva deleted the payment-element-renderer-lazy-removed branch September 17, 2024 11:03
@github-actions github-actions bot added Closed Label will be automatically added when the PR will get merged to main and removed Ready for Review PR with label Ready for Review should only be reviewed. labels Sep 17, 2024
akash-c-k pushed a commit that referenced this pull request Sep 17, 2024
## [0.87.2](v0.87.1...v0.87.2) (2024-09-17)

### Bug Fixes

* chunk loading issue - 1 ([#623](#623)) ([06e7fab](06e7fab))
@PritishBudhiraja PritishBudhiraja linked an issue Sep 17, 2024 that may be closed by this pull request
seekshiva added a commit that referenced this pull request Sep 20, 2024
akash-c-k pushed a commit that referenced this pull request Sep 20, 2024
## [0.88.2](v0.88.1...v0.88.2) (2024-09-20)

### Reverts

* chunk loading issue - 1 ([#623](#623)) ([#631](#631)) ([f719ff5](f719ff5))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Label will be automatically added when the PR will get merged to main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove Lazy Loading from PaymentElement
2 participants