-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Code Issue in /plugins/notifications/sendgrid #7068 #7115
base: develop
Are you sure you want to change the base?
Conversation
|
@tesfamickal is attempting to deploy a commit to the medusajs Team on Vercel. A member of the Team first needs to authorize it. |
Yes I found this issue too. Took me a day to figure this out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tesfamickal can I get you to rebase to the v1.x
branch?
@@ -686,7 +686,7 @@ You don’t have to create a template for every type in the reference. You can s | |||
|
|||
### Order Shipment Created | |||
|
|||
**Key in plugin options:** `order_shipped_template` | |||
**Key in plugin options:** `order_shipment_created_template` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
**Key in plugin options:** `order_shipment_created_template` | |
**Key in plugin options:** `order_shipment_created_template` in the root options, and `order_shipped_template` in the localization options. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shahednasser sure thing
changed order_shipped_template to be renamed to order_shipment_created_template