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

Annotations required updated #722

Merged
merged 4 commits into from
Aug 9, 2024
Merged

Annotations required updated #722

merged 4 commits into from
Aug 9, 2024

Conversation

kathryn-ods
Copy link
Contributor

Overview

What does this pull request do? fixes #721

How can a reviewer test or examine your changes?
check all the tests pass, build the docs and check the expected fields now show as required

Who is best placed to review it?

Closes #

Translations

  • New or edited strings appearing as a result of this PR will be picked up for translation
  • I've notified the translation coordinator of any new strings that will need
    translating. See the Handbook

Documentation & Release

@kathryn-ods
Copy link
Contributor Author

@kd-ods I think this ok to merge as won't actually impact the 0.4.0 branch but tagging for a sense check when you're back. Also should we be merging this in to 0.4.0 or 0.4.1 maybe?

@kathryn-ods kathryn-ods merged commit 8c2d888 into main Aug 9, 2024
4 checks passed
@kathryn-ods kathryn-ods deleted the annotations branch August 9, 2024 13:33
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.

Annotations required fields
2 participants