Updated marketing page and support for registration

[devyoustice]

This is submitted for early feedback for validation and approval. NOTE: Don't merge this release until further notice.

Special questions:

• can/should we use embedded fonts (as it is currently)?

Known limitations:

• The registration form won't create real accounts

[devyoustice]

Any feedback is really appreciated. The sooner, the better.

[Quetzacoalt91]

Hi @devyoustice,

Including new fonts is allowed. Just take care to store them in the folder views/fonts as you did and add an index.php file. Also, make sure they work in HTTP and HTTPS.

[Quetzacoalt91]

Your class Translator seems to be a duplicate from the translation system of PrestaShop. You can set your Youtube URLs as a translated text. Here is an example: https://github.com/PrestaShop/gadwords/blob/master/fr.php#L7

Also, describing why you need all this data during the registration would be appreciated.

cc @sdesbrosses

[devyoustice]

Those data are collected essential to carry out the registration. We will add some explanation in a future version, but for now let's do without. As for the translator class, yes it is pretty much a duplicate. The reason why it's part of the SDK is that we need it in other environments, or custom integrations and would like to minimize the differences between the generic SDK and the one included in the this module. Is this OK? URLS: OK, will be set as translatable text.

[devyoustice]

Commit https://github.com/devyoustice/yousticeresolutionsystem/commit/af6fe829b068222836d80191dcb09e72ce43a606 is mergeable and publishable.

[devyoustice]

Hi, could you please give any eta, when our submission is going to get reviewed?

[Quetzacoalt91]

Hi,

I plan to review modules tomorrow morning. You should have some news at this time.

Best regards

[devyoustice]

Thanks. Does it mean, that (if successfully validated), the module will immediately be available and published by you, or is that a separate process?

[Quetzacoalt91]

It is a separate process executed each night.

[devyoustice]

hi, any news?

[devyoustice]

Would it be an acceptable compromise this time? We will resolve it later, however, in due time.

[Quetzacoalt91]

We will see if you change it next time :smile:

[devyoustice]

Question: if we added the translation tags, it would show up in the translation UI, meaning, translators (person) would mistakenly assume, that these texts also would need to be translated into other languages, which, however, is not the case, and thus we can save translation costs and avoid misunderstandings.

[Quetzacoalt91]

Having this {if $languageCode == 'fr'} is in my opinion a temporary fix for displaying content in your views. That's why I consider this part will be removed later, which makes this translation tags mandatory.

Best regards

[devyoustice]

that is correct. Thanks for the clarification.