-
Notifications
You must be signed in to change notification settings - Fork 95
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
Removed the OrchardCore clone dependency to make this repo stand alone #65
Conversation
…tion folder. Changed Orchard Core references from project references to NuGet package references. Added a package reference to OrchardCore.Module.Targets.
Removed the OrchardCore clone dependency from the "Setting up your dev environment" instructions in the readme
What prevents you from using the rc2 version? .gitignore should probably exclude the .po files. A specific Setup recipe could enable some required features and execute the Product.recipe.json to create the Product content type (can be done in another PR). |
@agriffard I have ignored the .po files and updated the package references to rc2. The reason it was rc1 was because I used MyGet instead of NuGet because the readme said to use the dev branch of OrchardCore and I thought MyGet was where the dev branch was published to based on the www.ideliverable.com/blog/getting-started-with-orchard-core-as-a-nuget-package post which was linked to in https://docs.orchardcore.net/en/dev/docs/getting-started/ . |
Looks like the po files are still there. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good except for the po and .git* files. Thanks!
Thanks, this looks great. I'll merge it now, and I'll play with it tonight. Then, once I'm happy with the stability of the branch, I'll merge into main so others can use the new project structure as the basis for new code. |
Something's wrong. Can't get past setup:
|
The test project is also broken:
|
I will look into those. |
I'm not getting the same error as you got when running the site. I was able to setup the site (used the agency theme), enable the 3 Commerce features, add a product content type (with product, price, and title parts), add a test product, and view the product page. But I get this error when I try to add the test product to the cart:
|
I got past the
|
I can take care of the add to cart errors, they look unrelated to your changes. |
Do you want me to add the |
No, I'll take care of that as well, thanks. I thought you'd only need the xunit test adapter package, I need to check what's going on. |
Resolved this issue:
Build against Orchard packages instead of project references #64