Skip to content
This repository was archived by the owner on Jan 1, 2020. It is now read-only.

[WIP] Better support of content creation - #45

Merged
flack merged 6 commits into
openpsa:masterfrom
symfony-cmf:cmf
Jan 9, 2013
Merged

[WIP] Better support of content creation#45
flack merged 6 commits into
openpsa:masterfrom
symfony-cmf:cmf

Conversation

@adou600

@adou600 adou600 commented Dec 21, 2012

Copy link
Copy Markdown
Contributor

Minor additions are needed in createphp in order to be able to create routes documents.

This PR is linked to symfony-cmf/cmf-sandbox#143 and symfony-cmf/create-bundle#24.

Do not merge this PR for now, it has been created to synchronize the work in progress.

@flack

flack commented Dec 21, 2012

Copy link
Copy Markdown
Collaborator

well, I think for the AbstractMapper it's not really generic enough. But couldn't this code be in BaseDoctrineRdfMapper just as easily?

@flack

flack commented Jan 3, 2013

Copy link
Copy Markdown
Collaborator

I just noticed that I no longer get notifications for this PR for some reason, but I don't really have much to add here anyways :-) From my POV, everything looks good, so if you want me to merge, just let me know!

@dbu

dbu commented Jan 9, 2013

Copy link
Copy Markdown
Collaborator

@flack i think it looks good too. just talked with @adou600, we suggest to merge and if we find other issues while working on the cmf we open new PRs.

@flack

flack commented Jan 9, 2013

Copy link
Copy Markdown
Collaborator

Alright, merging as requested. BTW, just a small heads-up: I will be on vacation for the rest of January without real network access, so if there's anything else that is urgent, better say it now :-)

flack added a commit that referenced this pull request Jan 9, 2013
[WIP] Better support of content creation
@flack
flack merged commit 521a05b into openpsa:master Jan 9, 2013
@adou600

adou600 commented Jan 10, 2013

Copy link
Copy Markdown
Contributor Author

Great, thanks for the collaboration, and enjoy your vacations!

@dbu
dbu deleted the cmf branch December 20, 2013 15:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants