Skip to content

Conversation

@FredPeal
Copy link
Contributor

General Description

Please include a summary of the changes and the related issue. Highlight key points of the PR, any relevant information, and reasons for making the change.

Related Issue

A link to the ticket or issue that this PR is related to.(if applicable)

Checklist

A checklist of things that should be done before merging this PR. Linked to the ticket or issue that this PR is related to.(if applicable)

  • I have performed a self-review of my code.
  • I have added tests to cover my changes.
  • I have updated the documentation (if needed).
  • My changes generate no new warnings.
  • My changes do not break any existing tests.

Screenshots (if applicable)

Provide screenshots of impactful changes you have made so that the reviewer can quickly understand the changes.

@sentry
Copy link

sentry bot commented Apr 10, 2025

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: src/Domains/Connectors/CMLink/Actions/CreateEsimOrderAction.php

Function Unhandled Issue
Kanvas\Connectors\CMLink\Actions\CreateEsimOrderAction::execute Kanvas\Exceptions\ValidationException: Order already has eSim metadata /src/Domains/Connectors/CMLink/Actions/CreateEsimOrderAction.php in Kanvas\Connectors\CMLink\Actions\CreateEsimOrde...
Event Count: 7

Did you find this useful? React with a 👍 or 👎

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.

3 participants