-
Notifications
You must be signed in to change notification settings - Fork 94
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
NEST-537: Upgrading to OC 2.1.0 #512
Conversation
Co-authored-by: Mike Alhayek <mike@crestapps.com> Co-authored-by: Zoltán Lehóczky <zoltan.lehoczky@lombiq.com>
* Upgrade OC packages and drivers * Update src/Modules/OrchardCore.Commerce/Drivers/PriceDisplaySettingsDisplayDriver.cs Co-authored-by: Sára El-Saig <sara.el-saig@lombiq.com> * Update src/Modules/OrchardCore.Commerce.ContentFields/Settings/AddressFieldSettingsDriver.cs Co-authored-by: Sára El-Saig <sara.el-saig@lombiq.com> * Update src/Modules/OrchardCore.Commerce.ContentFields/Settings/PriceFieldSettingsDriver.cs Co-authored-by: Sára El-Saig <sara.el-saig@lombiq.com> * Update src/Modules/OrchardCore.Commerce.Payment.Stripe/Drivers/StripeApiSettingsDisplayDriver.cs Co-authored-by: Sára El-Saig <sara.el-saig@lombiq.com> * Fix all drivers --------- Co-authored-by: Sára El-Saig <sara.el-saig@lombiq.com>
…OrchardCMS/OrchardCore.Commerce into task/system-text-json-migration
* Update OC version. * Update OC preview version again (sigh). * Delete temporary extension. * Use the new AddDeployment extension that registers the underlying polymorphic JSON type too. * Suppress. * Update OC previews. * Update OC preview version. * Update OC preview version. * Rename to localizer. * Update OC preview version. * Update OC preview version. * Add new line --------- Co-authored-by: Hisham Bin Ateya <hishamco_2007@yahoo.com>
# Conflicts: # Directory.Packages.props # docs/releases/3.0.0.md # src/Modules/OrchardCore.Commerce.Payment.Stripe/Controllers/StripeController.cs
how about the situation of this PR. Can we just use the code until |
It's work in progress, but hoping to finish the whole thing (this PR is part of a larger task with a dozen or so PRs) next week. Aside from the security scanning test failing, this looks stable. If you can give it a try and provide feedback, that would be awesome! |
…Scan extension method
… by UseAssertAppLogsForSecurityScan
This pull request has merge conflicts. Please resolve those before requesting a review. |
NEST-537