Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add application cpm_data to separate data from site structure #85

Merged
merged 9 commits into from
Dec 9, 2016

Conversation

staaas
Copy link
Collaborator

@staaas staaas commented Dec 8, 2016

This adds cpm_data application to separate data from site structure. My final goal is to get rid of Page objects that don't represent public site pages.

This was inspired by site menu (developed by @abitrolly).
modeladmin

wagtail.contrib.modeladmin provides some tools for making independent admin-like CRUD interfaces (that still look like Wagtail). Looks like this module miss a lot of things. The main one is a generalized chooser panel (so I had to implement one). Also I miss Page-like revisions/publications.

modeladminutils contains our enahncements for wagtail.contrib.modeladmin. I'll probably try to push them back to wagtail.contrib.modeladmin some time later.

@staaas
Copy link
Collaborator Author

staaas commented Dec 8, 2016

@abitrolly @davojta could you please have a look?

from modeladminutils.widgets import GenericModelChooser


class BaseGenericModelChooserPanel(BaseChooserPanel):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need separate panel and cant just use http://docs.wagtail.io/en/v1.7/topics/snippets.html for our models ?

Copy link
Collaborator Author

@staaas staaas Dec 8, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i doubt we can use snippets panel without actually using snippets

snippets is an easier approach
i think i should have used it instead of implementing this panel

but now when my widget is implemented, modeladmin approach opens more opportunities

@staaas
Copy link
Collaborator Author

staaas commented Dec 8, 2016

@abitrolly is deploy/netlify failure expected?

@staaas
Copy link
Collaborator Author

staaas commented Dec 8, 2016

@davojta is this gtg? once we merge this i'll start converting data from results

@davojta
Copy link
Member

davojta commented Dec 8, 2016 via email

@staaas
Copy link
Collaborator Author

staaas commented Dec 9, 2016

@davojta could you please approve this then? :)

Copy link
Member

@davojta davojta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@staaas staaas merged commit 3999498 into kinaklub:master Dec 9, 2016
@staaas staaas deleted the cpm-data branch December 9, 2016 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants