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 email template creation date not being persisted #859

Merged
merged 1 commit into from
Dec 25, 2014

Conversation

kabel
Copy link
Contributor

@kabel kabel commented Dec 22, 2014

The database schema from https://github.com/magento/magento2/blob/develop/app/code/Magento/Email/sql/email_setup/install-2.0.0.php#L62-L67 uses the column added_at for storing the date the email template creation date. When setCreatedAt is used, that date is not persisted to the database, leaving the column NULL in the database.

The database schema from https://github.com/magento/magento2/blob/develop/app/code/Magento/Email/sql/email_setup/install-2.0.0.php#L62-L67 uses the column `added_at` for storing the date the email template creation date. When `setCreatedAt` is used, that date is not persisted to the database, leaving the column `NULL` in the database.
@maksek
Copy link
Contributor

maksek commented Dec 25, 2014

Code Review passed
Builds green
Resolution: OK to merge

@maksek maksek added the PS label Dec 25, 2014
@maksek maksek self-assigned this Dec 25, 2014
maksek pushed a commit that referenced this pull request Dec 25, 2014
Fix email template creation date not being persisted (MAGETWO-32152)
@maksek maksek merged commit 7e77673 into magento:develop Dec 25, 2014
@kabel kabel deleted the patch-2 branch August 10, 2015 04:06
mmansoor-magento pushed a commit that referenced this pull request Feb 21, 2017
[Mavericks] Extend functional tests coverage

- Tasks:
- MTA-3977: Create auto test for Checkout using PayPal Braintree button if Require Customer's Billing Address = Yes
- MTA-4065: Import Advanced Pricing if Incorrect Data
magento-engcom-team added a commit that referenced this pull request Aug 29, 2019
…bleProductGra… #859

 - Merge Pull Request magento/graphql-ce#859 from danslo/graphql-ce:configurable-product-graph-ql-dependency
 - Merged commits:
   1. ccf86d3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants