- Issue created by @Dobefu
- Merge request !33437140 - Merge the separate "preview_token" database field into the "value" database field → (Merged) created by Dobefu
- Status changed to Needs review
3 months ago 3:34pm 30 March 2024 - First commit to issue fork.
- 🇦🇺Australia AlMunnings Melbourne
almunnings → changed the visibility of the branch 3437140-ensure-table-exists to hidden.
- 🇦🇺Australia AlMunnings Melbourne
Howdy @Dobefu,
I've provided an alternative MR that may address the issue, without changing the current API.Are you able to test please?
Thank you
- 🇦🇺Australia AlMunnings Melbourne
I actually prefer your changes :D
I'll merge into dev, as the MR went a little haywire.LGTM. 🌮
- Status changed to Fixed
3 months ago 3:05am 7 April 2024 - 🇳🇱Netherlands Dobefu
Hiya @almunnings,
Thanks for the response and the quick merge! I've updated to the latest released version, and have done some tests.
Installation works fine, as does updating from the previous version. Preview tokens also get generated and read correctly.Since the changes have been included in a new release, I'll take the liberty of closing this issue :D
Thanks again!
- Status changed to Fixed
3 months ago 10:17am 7 April 2024 - 🇦🇺Australia AlMunnings Melbourne
Added some extra code on dev, I want to really validate that token.
Added escapeToken method
And load up/compare the token after the LIKE search.Paranoia.