Skip to content

Conversation

@mikl
Copy link
Member

@mikl mikl commented Mar 21, 2012

Instead of being a part of ding_content.

@kasperg
Copy link

kasperg commented Mar 22, 2012

Nice refactoring!

@mikl In reference to your comment on Skype: Is this ready for review?

@mikl
Copy link
Member Author

mikl commented Mar 22, 2012

I still need to write an upgrade hook for ding_content that enables the ding_staff_profile module, but otherwise, yes.

@mikl
Copy link
Member Author

mikl commented Mar 28, 2012

@kasperg this should be ready to go now :)

@mikl
Copy link
Member Author

mikl commented Aug 9, 2012

@kasperg any chance this will be merged any time soon?

@mikl mikl mentioned this pull request Aug 9, 2012
@cableman
Copy link
Member

Reviewed and approved

cableman added a commit that referenced this pull request Oct 15, 2012
Split out ding_staff_profile as a separate module…
@cableman cableman merged commit e308aa9 into dingproject:master Oct 15, 2012
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.

3 participants