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

Fix single quotes #490

Merged
merged 3 commits into from
Aug 31, 2023
Merged

Fix single quotes #490

merged 3 commits into from
Aug 31, 2023

Conversation

yvanzo
Copy link
Contributor

@yvanzo yvanzo commented Aug 31, 2023

Problem

An ISE (500) was returned instead of the page “About” with the pull requests #488 and #489 not released yet.

Solution

  • Replace curved single quotes ‘’ that were used instead of straight one ' as string delimiter

Extra

  • Replace straight single quotes ' that were used instead of typographic apostrophe in strings
  • Replace straight single quotes ' that were used instead of curved quotation marks “” in strings

yvanzo added 3 commits August 31, 2023 12:26
Curved quotation marks were used instead of straight ones as Python
string delimiters, whereas they should be used inside strings only.

That broke returning the “About” page.

This patch fixes delimiters and escaped occurrences.
Replace straight single quotes in strings where appropriate.
Replace straight single quotes in strings where appropriate.
@github-actions

This comment has been minimized.

@yvanzo
Copy link
Contributor Author

yvanzo commented Aug 31, 2023

Manually tested in beta the pages that have been changed only.
CI test failures are unrelated, there is another pull request opened about those.

@yvanzo yvanzo merged commit 3c89596 into master Aug 31, 2023
@yvanzo yvanzo deleted the fix-single-quotes branch August 31, 2023 12:09
@github-actions
Copy link

Unit Test Results

    1 files  ±0      1 suites  ±0   9m 11s ⏱️ ±0s
201 tests ±0  199 ✔️ ±0  0 💤 ±0  2 ❌ ±0 

For more details on these failures, see this check.

Results for commit 3c89596. ± Comparison against base commit 3c89596.

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.

1 participant