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

PDF- ToolKIT 6.2 upgrade #64

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

Conversation

Madushani
Copy link

No description provided.

@douglascrp
Copy link
Contributor

Hello.

Thanks for taking the time to work on this, but I have to ask... Have you tried the latest code version before doing the upgrade by yourself?
As you can see in #55, the code has already been changed to work with Alfresco 6.x, so I believe you should be trying it and check it there is any kind of incompatibility with the version 6.2.

Besides that, I see you have created a really big PR, with so much changes (basically replacing all the existing code), that it seem really really hard to understand what has been changed.

Can you review that?

@Madushani
Copy link
Author

Hello douglascrp,

Sorry for the confusion. I have made a new pull request. It contains the destination folder UI change

@nachopants
Copy link

nachopants commented Feb 25, 2020

I can confirm the latest release does not work with v6.2 community, or the latest ECM enterprise.

Tried installing on both with fresh deployments and no other plugins.

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