magento-2-smtp icon indicating copy to clipboard operation
magento-2-smtp copied to clipboard

Fix incorrect field type for `mp_smtp_ace_token`

Open timoffmax opened this issue 5 months ago • 0 comments

Description

Field mp_smtp_ace_token of the quote table currently has type smallint, so it can't handle the generated token. It looks like when the code was switching from the setup scripts to the DB Schema approach, the field type was accidentally set incorrectly.

Here we can see that the content that is put into the field should be a string: \Mageplaza\Smtp\Observer\Quote\SetToken.

This PR reverts the mp_smtp_ace_token field type back to varchar 255.

Related Pull Requests

No related PR were found.

Fixed Issues (if relevant)

No open issues were found.

Manual testing scenarios

  1. Open the quote table
  2. Check the field type
  3. Check the data in the column
  4. Place an order and make sure that mp_smtp_ace_token is generated correctly for your quote

Questions or comments

No extra questions or comments.

timoffmax avatar Sep 03 '24 07:09 timoffmax