Skip to content
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

Documentation for custom tags #1664

Merged
merged 1 commit into from
Dec 28, 2023
Merged

Conversation

andrew-3
Copy link
Contributor

No description provided.

@diana-lakatos diana-lakatos requested a review from piotrze October 13, 2023 07:04
Copy link
Contributor

@piotrze piotrze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not use {% form %} tag.

return_to: /success_page
---

{% form html-novalidate: true, html-class: 'model-form' %}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andrew-3 @diana-lakatos {% form %} tag is deprecated. Instead we should provide example with page that have normal html <form>.
Inside

you can use spam_protection tag or you can enter own code for generating hcaptcha code (https://github.com/Platform-OS/product-marketplace-template/blob/master/app/views/partials/hcaptcha.liquid)
In order to validate hcaptcha you can use this function https://github.com/Platform-OS/product-marketplace-template/blob/master/app/views/partials/lib/validations/hcaptcha.liquid

Please do not add examples that use {% form %} tag. We should change all examples that use it.

Copy link
Contributor

@piotrze piotrze Nov 10, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@diana-lakatos I've created ticket for it in past, sadly it was closed.....
#1497

@andrew-3 andrew-3 force-pushed the documentation-for-custom-tags branch from bbabbea to 358c137 Compare November 10, 2023 22:39
@andrew-3
Copy link
Contributor Author

@piotrze I've removed from the page those which were only form-specific, and converted the others from the form tag to HTML tags.

Thanks!

@Slashek Slashek merged commit 1218e8e into master Dec 28, 2023
@Slashek Slashek deleted the documentation-for-custom-tags branch December 28, 2023 09:20
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.

3 participants