Dan Braghiș

Results 382 comments of Dan Braghiș
trafficstars

Hey all, happy to review a pull request with the fix noted above

I find it odd that it fails silently 🤔 Perhaps we're not doing appropriate cleanup. If you happened to dig into this more, any further details will be greatly appreciated.

On using a fork.. you cannot really do it because of the FE assets.. best to build it locally and host the built package somewhere you can install from a...

Hey @hpoul, [`SynchronisedField`](https://www.wagtail-localize.org/ref/translatable-fields/#wagtail_localize.fields.SynchronizedField), or "overridable segment", does show in the UI. By default it syncs the value from the source, but the value can be overridden. See https://www.wagtail-localize.org/concept/segments/#overridable-segment

@zemogle this PR is still open, so it did not, I'm afraid

@enzedonline can you double check, is EmailSignature a site setting (which we don't yet support), or a snippet?

Hey @hughrawlinson, https://github.com/wagtail/wagtail-bakery/blob/master/tests/integration/test_views.py#L27 is probably the closest to emulate. You want to call https://github.com/wagtail/wagtail-bakery/blob/c63597ac405c3c35bc0b3206c58d6e5d7b9be6c8/src/wagtailbakery/views.py#L84 and probably mock `WagtailBakeryView.build_file` to raise an exception, then `override_settings` with different values for `BAKERY_IGNORE_OBJECTS_THAT_FAILED_TO_BUILD`

The task really is to submit a PR to django-bakery https://github.com/palewire/django-bakery/issues/160

Thank you for the tweaks @rosscdh. It is great to have a test. I went on to replicate the issue locally using the [examples/multisite](https://github.com/wagtail/wagtail-bakery/tree/master/examples/multisite) project provided with wagtail-bakery, with ```python...