Skip to content

Fix duplicated word in default exception template#3053

Open
xcompass wants to merge 1 commit into
openwebwork:WeBWorK-2.21from
ubc:upstream-exception-template-what-what-typo
Open

Fix duplicated word in default exception template#3053
xcompass wants to merge 1 commit into
openwebwork:WeBWorK-2.21from
ubc:upstream-exception-template-what-what-typo

Conversation

@xcompass

Copy link
Copy Markdown
Member

Summary

Fixes a duplicated word in templates/exception_default.html.ep. The paragraph asking users to report errors to the site's webmaster currently reads:

... including all of the following information as well as what what you were doing when the error occurred.

This removes the duplicated "what" so the sentence reads correctly. No functional or markup changes — a one-word text fix in the default exception page shown to end users when an unhandled error occurs.

Testing

  • Verified the duplicated word is present in the current WeBWorK-2.21 branch (and develop) and that the diff is limited to removing the single extra "what" on that line (one line changed, nothing else).
  • Confirmed the file is a Mojolicious .html.ep template, which is not covered by the repository's perltidy (*.pl/*.pm/*.t/*.at) or prettier (htdocs/**/*.{js,css,scss,html}) format checks, so no formatting tooling applies.

Dependencies: none

The paragraph asking users to report errors to the site's webmaster
read "as well as what what you were doing when the error occurred".
Remove the duplicated "what".
@xcompass

Copy link
Copy Markdown
Member Author

Sending a few PRs from our local improvements. Happy to switch the target branch to develop if needed.

@drgrice1 drgrice1 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good. Could you please shorten what you say when you make these pull requests? Is it generated by AI? There is a lot of wording when all that is needed would have been "Fix a typo".

@xcompass

Copy link
Copy Markdown
Member Author

Will do.

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