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

Don't truncate in array item restrictions tables #216

Merged

Conversation

alexjurkiewicz
Copy link
Contributor

The table "Each item of this array must be" contains a list of valid items, along with (the first line of) their description. Don't truncate the length of this description. It can break formatting if the description itself is written in Markdown.

The modified first_line_fixed function is also exported to Jinja, but is apparently unused. So don't do that either.

Example of table before this change:

Each item of this array must be Description
Privilege [MySQL privilege](https://dev.mysql.com/doc/refman/8.0/en/grant.html#grant-privi ...

Example after this change:

Each item of this array must be Description
Privilege MySQL privilege to grant.

The table "Each item of this array must be" contains a list of valid
items, along with (the first line of) their description. Don't truncate
the length of this description. It can break formatting if the
description itself is written in Markdown.

The modified first_line_fixed function is also exported to Jinja, but is
apparently unused. So don't do that either.

Example of table before this change:

| Each item of this array must be                      | Description                                                                          |
| ---------------------------------------------------- | ------------------------------------------------------------------------------------ |
| [Privilege](#rds_items_users_items_privileges_items) | [MySQL privilege](https://dev.mysql.com/doc/refman/8.0/en/grant.html#grant-privi ... |

Example after this change:

| Each item of this array must be                      | Description                                                                          |
| ---------------------------------------------------- | ------------------------------------------------------------------------------------------------ |
| [Privilege](#rds_items_users_items_privileges_items) | [MySQL privilege](https://dev.mysql.com/doc/refman/8.0/en/grant.html#grant-privileges) to grant. |
@dblanchette dblanchette merged commit 178c7b1 into coveooss:main Sep 28, 2023
5 checks passed
@dblanchette
Copy link
Collaborator

Thanks!

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