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: isend sms provider unexpected provider error #5135

Merged
merged 7 commits into from
Feb 3, 2024
Merged

Conversation

jainpawan21
Copy link
Member

@jainpawan21 jainpawan21 commented Jan 30, 2024

What change does this PR introduce?

  1. Change png image to SVG image
  2. Add apiToken in credential schema and dto
  3. Add error name and message in execution details log

Why was this change needed?

closes #5123

Other information (Screenshots)

Copy link

linear bot commented Jan 30, 2024

Copy link

netlify bot commented Jan 30, 2024

Deploy Preview for dev-web-novu failed.

Name Link
🔨 Latest commit 28a895d
🔍 Latest deploy log https://app.netlify.com/sites/dev-web-novu/deploys/65be832e90ef94000857f765

@jainpawan21 jainpawan21 requested a review from LetItRock January 30, 2024 19:22
package.json Outdated
@@ -13,7 +13,7 @@
"commit": "cz",
"nx": "nx",
"lint-staged": "lint-staged",
"generate:provider": "npx hygen provider new --version 0.16.3",
"generate:provider": "npx hygen provider new --version 0.22.0",
Copy link
Member

Choose a reason for hiding this comment

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

Nice 👌🏼
But I am wondering if there is a way we can automate this version number.

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure how to read version in package.json 🙂

Copy link
Member

Choose a reason for hiding this comment

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

Let's create a linear ticket for that and address it later

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

@BiswaViraj BiswaViraj left a comment

Choose a reason for hiding this comment

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

Please add the provider to the sort-providers file on web

@jainpawan21
Copy link
Member Author

Please add the provider to the sort-providers file on web

Added ✅

@jainpawan21 jainpawan21 merged commit 033b90d into next Feb 3, 2024
23 of 30 checks passed
@jainpawan21 jainpawan21 deleted the NV-3466 branch February 3, 2024 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants