Skip to content

Added release blurb and fixed grammar in api breaking sentence - #12376

Merged
feerrenrut merged 4 commits into
masterfrom
AddReleaseBlurb
May 13, 2021
Merged

Added release blurb and fixed grammar in api breaking sentence#12376
feerrenrut merged 4 commits into
masterfrom
AddReleaseBlurb

Conversation

@Qchristensen

@Qchristensen Qchristensen commented May 7, 2021

Copy link
Copy Markdown
Member

Link to issue number:

Summary of the issue:

Needed a release blurb for 2021.1 and also there was a grammar error in the "API breaking sentence" in the what's new document.

Description of how this pull request fixes the issue:

With words

Testing strategy:

Read text, and ran past others when writing.

Known issues with pull request:

Change log entries:

For Developers

Code Review Checklist:

  • Pull Request description is up to date.
  • Unit tests.
  • System (end to end) tests.
  • Manual tests.
  • User Documentation.
  • Change log entry.
  • Context sensitive help for GUI changes.

@josephsl

josephsl commented May 7, 2021

Copy link
Copy Markdown
Contributor

Hi,

In the release blurb: I think there is still a grammatical error with "add-on API" sentence, and may I suggest splitting that note into a separate line/paragraph?

Thanks.

Comment thread user_docs/en/changes.t2t Outdated
Comment thread user_docs/en/changes.t2t Outdated
Qchristensen and others added 2 commits May 12, 2021 09:19
Thanks Reef!

Co-authored-by: Reef Turner <feerrenrut@users.noreply.github.com>
Updated "maths" to "mathematics" as per Bill's request - thanks Bill!

Also updated the wording of one of the bug fixes to fix the grammar.  Changed from "... to use NVDA in a languages ..." to "... to use NVDA in languages ..."
(Thanks Susanne via email for pointing that one out)
@AppVeyorBot

Copy link
Copy Markdown

@seanbudd seanbudd added this to the 2021.1 milestone May 12, 2021
Comment thread user_docs/en/changes.t2t Outdated
Updated "Mathematics" to "Mathematical"

Co-authored-by: Bill Dengler <codeofdusk@gmail.com>

@feerrenrut feerrenrut left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good @Qchristensen

@feerrenrut
feerrenrut marked this pull request as ready for review May 13, 2021 06:08
@feerrenrut
feerrenrut requested a review from a team as a code owner May 13, 2021 06:08
@feerrenrut
feerrenrut requested a review from seanbudd May 13, 2021 06:08
@feerrenrut
feerrenrut merged commit ea1b827 into master May 13, 2021
@feerrenrut
feerrenrut deleted the AddReleaseBlurb branch May 13, 2021 06:10
@CyrilleB79 CyrilleB79 mentioned this pull request May 26, 2021
8 tasks
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.

6 participants