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

Migrate to OrchardCore 2.0 #130

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

asimeonov
Copy link

@asimeonov asimeonov commented Sep 25, 2024

Hi everyone. I made an update to OC 2.0. Can you please review this PR and eventually merge it. We have migrated our software to latest version that was not compatible with the Nuget package. We have tested all of the fields that we use locally and so far we have no issues with the implementation. However we don't use all of the Fields provided so your review and testing will be much appreciated.

@vengi83644
Copy link

Our upgrade to the latest OC 2.0 version is blocked by this. Is this something that I can help with?

@asimeonov
Copy link
Author

asimeonov commented Dec 2, 2024

Our upgrade to the latest OC 2.0 version is blocked by this. Is this something that I can help with?

Same with us, this is why I have prepared this update. No one from Etch team have reviewed this so far 😞. As an alternative you can checkout this branch and add the project to your solution instead of a reference. This is what we have did.
Can you please help? @peterkeating @michaelchart @lawrencedine

@peterkeating
Copy link
Member

@asimeonov Thanks so much for your PR, I'll check it over and look to get it deployed this week.

@asimeonov
Copy link
Author

@asimeonov Thanks so much for your PR, I'll check it over and look to get it deployed this week.

I have updated to latest OC version, due to bug fixes implemented after 2.0.0. We have tested the update already on our system, please have another look.

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