I’ve spent some time this week testing lilac.master via the native installation, and while I couldn’t find any showstoppers on the basic features (including the Account MFE! Thanks, @jhony_avella!), we’re still missing proper support for the Payment and Ecommerce MFEs.
Which is to say, thanks to @Maksim_Sokolskiy’s work (thanks!), the PR is there and I can confirm it works after backporting it to lilac.master… but it is not in as plug-and-play a state as, for instance, the Account MFE. Which is to say, following the native installation steps does not get one a working ecommerce installation, even though backend and MFEs are correctly deployed.
This is no worse than Koa or previous releases. We’ve always had to go the extra mile to get ecommerce properly configured. But this brings me to the titular question of this post:
As I see it, we have a few options:
- Release it immediately…
a. … without ecommerce/payment MFE support at all.
b. … with @Maksim_Sokolskiy’s changes, even if they don’t make it into master.
- Wait for:
a. The PR to be merged to master and backported to Lilac
b. A second PR that either adds documentation on how to enable a minimally testable payment/ecommerce or makes it so the user gets them with minimal effort
Thoughts? I’m of the opinion that we should go with 1.b, and make an RC on Monday with that PR, merged to master or not. This will give prospective testers the full featureset to test, even if not in a final state. Feel free to disagree, though.