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

make available to import to dexterity blob types #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

maartenkling
Copy link

No description provided.

@maartenkling
Copy link
Author

Garbas, could you review my pull request? i like to get it out of my list :)

@idgserpro
Copy link
Member

@maartenkling collective.blueprint.jsonmigrator was "renamed" to collective.jsonmigrator.

Would you mind:

  • Checking collective.jsonmigrator source code and see if something similar to your pull was done there;
  • If nothing similar was done, redo the PR in collective.jsonmigrator.

...since the plans for collective.blueprint.jsonmigrator is to make it inactive?

@maartenkling
Copy link
Author

@idgserpro
i have no time, the pullrequest is for dexteritycontent if you need it use it :)

@djowett
Copy link
Contributor

djowett commented Oct 26, 2015

@idgserpro I've made an issue in collective.jsonmigrator, so we can handle this ourselves there.
Btw we can't make this product inactive, only recommend alternatives - it's open source after all!

@idgserpro
Copy link
Member

Thanks @djowett. I think this pull can be closed if @garbas is ok with that. If this package is being superceeded by collective.jsonmigrator, IMHO all activity here should be ceased. I know we can't "make" this inactive, since nothing stops someone from creating a fork from this package.

I still think it's a nice idea to wait for a response from @yboussard in his PR in #1.

@garbas
Copy link
Contributor

garbas commented Nov 30, 2015

@djowett @idgserpro @maartenkling as you probably see i'm not that active in github.com/collective anymore. please feel free to take the ownership of my packages if interested.

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.

4 participants