You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The InstallationService is massive, making local changes and synching is a pain (e.g. Metric UPS requires CM #7121).
Since the InstallationService is already a partial class, splitting it in three makes it easier to work with. An alternative (better?) way of doing this would be to inject two other services.
I've done this in my project here, but after I re-based I realised I'd missed some changes from @exileDev 21 hours ago, so maybe one of the core team should implement this.
The text was updated successfully, but these errors were encountered:
nopCommerce version:
Steps to reproduce the problem:
Since the InstallationService is already a partial class, splitting it in three makes it easier to work with. An alternative (better?) way of doing this would be to inject two other services.
I've done this in my project here, but after I re-based I realised I'd missed some changes from @exileDev 21 hours ago, so maybe one of the core team should implement this.
The text was updated successfully, but these errors were encountered: