Add a `wagtail_integration` integration app for Wagtail support about django-fobi HOT 18 CLOSED

@ababic:

Firstly - thanks for support!

It's preferred (remember: DRY). It worked fine with DjangoCMS, FeinCMS and Mezzanine integration.

Look, for instance here it's done as follows:

• https://github.com/barseghyanartur/django-fobi/blob/master/src/fobi/contrib/apps/djangocms_integration/cms_plugins.py#L42
• https://github.com/barseghyanartur/django-fobi/blob/master/src/fobi/contrib/apps/feincms_integration/widgets.py#L92
• https://github.com/barseghyanartur/django-fobi/blob/master/src/fobi/contrib/apps/mezzanine_integration/page_processors.py#L30

So, the internal method _process does everything for you.

But, if wagtail is different you are free to do it the way it's done in wagtail. I'll probably make some modifications later to make it as DRY as possible.

from django-fobi.

Fobi is pretty incredible. I'm also looking to use it with wagtail and am willing to help. It looks like this has stalled. What's the best way to get started?

from django-fobi.

@ababic:

Thank you! Your code was a good starting point.

from django-fobi.

@barseghyanartur I haven't had a chance yet, hoping to give it a go in the next couple of days.

from django-fobi.

So, I guess my first question is: Is it a requirement that form handling itself is done via fobi.integration.processors.IntegrationProcessor? And if so, what would be your preferred approach for adding additional items to the context?

from django-fobi.

Hi @barseghyanartur. Am i correct in thinking that a template chosen for the form_template_name field should just be responsible for rendering the 'form' part in a view, which can then be output anywhere in a custom template?

And does that differ for the success_page_template_name field? Is that template supposed to be responsible for rendering the entire page/view, or just part of one?

Thanks

from django-fobi.

@ababic:

Check this:

• https://github.com/barseghyanartur/django-fobi/blob/master/examples/simple/settings/base.py#L412
• https://github.com/barseghyanartur/django-fobi/tree/master/examples/simple/templates/fobi/bootstrap3_extras

from django-fobi.

@ababic Did you ever get anywhere with Wagtail integration? I'm hitting a wall with wagtailforms and this would be a great way to get around that.

from django-fobi.

@dwasyl:

FYI, contributions are much appreciated. I don't have wagtail experience and at the moment I'm too much busy with resurrection of another package of mine (django-dash), which I left unmanaged way too long. Thus, I don't have time for it yet (unless it arrives as a sponsored feature request).

In any case, work on wagtail integration shall be continued based on the wagtail/integration branch.

from django-fobi.

@gfranxman:

I'm gonna do a tiny release today and then I'll sync the wagtail/integration branch with master.

Do you have wagtail experience? Some work has already been done (although you may throw it all away, if you think it doesn't make sense).

Just as in case of DjangoCMS or FeinCMS, the contrib package should provide plugins for integration with CMS. It may have templates as well, although not necessarily.

There should be a working example. In case of DjangoCMS and FeinCMS I integrated it inside the simple project. In case of Mezzanine, the changes were too much, so I use a dedicated mezzanine_example project.

I seemed to me it makes sense to do the same with wagtail.

from django-fobi.

@gfranxman:

I have also included wagtaildemo integration project (as a git sub-module), which I started on about a half year ago.

from django-fobi.

@gfranxman:

FYI, I'm almost done with wagtail integration. Do you think you could take over the testing part?

from django-fobi.

from django-fobi.

@gfranxman:

The wagtail/integration branch seems to work for me.

See the git-submodule wagtaildemo.

Wagtail demo seems to be based on bootstrap3, so I used bootstrap3 theme there as well.

See the screen-shots below.

Fobi form page view

Fobi form submitted view

Wagtail admin - fobi form page edit view

Wagtail admin - create a page view

from django-fobi.

@ababic, @dwasyl:

Could you, guys, also do some tests?

from django-fobi.

Any remarks yet?

from django-fobi.

@barseghyanartur I've got too much on right now I'm afraid. Trying to get a lot of things tied up before I go on holiday.

from django-fobi.

I merged this branch into master. Version 0.12 has Wagtail support (yet experimental).

This ticket is closed. Feel free to open a new issue if needed.

from django-fobi.