mozilla / wagtail-localize-smartling

Integration between wagtail-localize and Smartling's API
Mozilla Public License 2.0
2 stars 1 forks source link

Pages with snippets included in them cause an error at the sync stage #40

Open stevejalim opened 1 day ago

stevejalim commented 1 day ago

(I'll add more context as I unearth it, but writing this down immediately)

When a Page has a Snippet associated with it, that Snippet does seem to get sent to Smartling, but as far as Wagtail is concerned it remains in unsynced, while the Page is synced

Page

Screenshot 2024-11-22 at 13 48 58

Snippet that features in the page

Screenshot 2024-11-22 at 13 48 53

Looking at the logs for the the sync, it appears that wagtail-localize-smartling doesn't remember that it's already sent the Snippet for translation, so tries to perform an initial sync, not an update sync

Screenshot 2024-11-22 at 13 57 26

stevejalim commented 1 day ago

Thinking about this, it's entirely possible that this is because w-l-s tried to send a visual context for the Snippet, which is not currently possible - and then that caused a failure at Job creation time...

stevejalim commented 1 day ago

Yeah i think that's what's up here

Screenshot 2024-11-22 at 15 01 12

stevejalim commented 1 day ago

On it