Skip to content

Conversation

@meiremans
Copy link
Contributor

@meiremans meiremans commented Aug 26, 2021

Maybe overloading you with PR's is not nice.

So this are all the changes bundled.

allows for payment with wiretransfer or pay on delivery
Added Romanian translation.
save the currency used in the order.
bumped stripe, as the current version didn't seem to work
bumped docker node image, as 8 is deprecated
added a fuzzy search with 1 edit as range for the search.
added missing I18N
make product data multilanguage
make menu multilanguage
productfeed will be made for every language
revised hook system & add product onCreate hook

Translation of everything will follow next week
Translation of everything will follow next week
Signed-off-by: nick <nick@ab-it.io>
there are shipping costs when using wiretransfer or pay on delivery
themes are loaded from views not public
current version didn't work for me, bumping it solved the issue
bumped it to 14. seems to work fine.
running on a deprecated LTS is not recomended?
Signed-off-by: nick <nick@ab-it.io>
fixed a lot of missing I18N fixed strings
Signed-off-by: nick <nick@ab-it.io>
Signed-off-by: Nick <nick@ab-it.io>
Signed-off-by: Nick <nick@ab-it.io>
Signed-off-by: Nick <nick@ab-it.io>
remove dev from default compose up command
no need to install extras, as we are not using alpine anymore
put back productId in hidden field (removed in merge)
will take thumbnail if one was generated in the system, otherwise it …
remove dimensions if one dimension is not filled in
add deploy to dockerfile to generate min js/css on rebuild
actually wait for promises to resolve
@meiremans
Copy link
Contributor Author

Hello.

This became a bunch of commits.
As for this version (V1.2.3 on my branch), I deployed a demo: https://expresscart.ab-it.io/
(same login as your demo).

Could you have a look, see if these are changes you would like to incorporate in expressCart?
If you think these changes are out of scope, I'll close the PR.

If you have some remarks, I'll be glad to solve them.
Thank you.

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.

2 participants