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

Response models #18

Draft
wants to merge 2 commits into
base: prompt-models
Choose a base branch
from
Draft

Response models #18

wants to merge 2 commits into from

Conversation

peterjwest
Copy link
Collaborator

Description of the change

TODO

Tickets

Dev checklist

  • PR addresses all ACs
  • Stay on point and keep it small so the merge request can be easily reviewed.
  • Added unit tests if necessary
  • Updated documentation if necessary
  • Self code review
  • Requested code review

Manual testing steps, if applicable

Please provide detail on the technical changes, if relevant

Screenshots/GIFs

Specific checklists

Unit tests
  • Added
Documentation
  • Updated build docs
  • Updated other technical docs such as tech spec or field spec
Browser testing
  • I have tested in the following browsers and environments (tick all that apply - not all changes will need testing in every browser)
    • Latest version of Chrome on mac
    • Latest version of Firefox on mac
    • Latest version of Safari on mac
    • Safari on last two versions of iOS
    • Chrome on last two versions of Android
Accessibility
  • Automated WCAG 2.1 tests pass
  • HTML validation passes
  • Manual WCAG 2.1 tests completed
  • I have tested in a screen reader
  • I have tested in high-contrast mode
  • Any animations removed for prefers-reduced-motion
Sustainability
  • Images are optimised and lazy-loading used where appropriate
  • SVGs have been optimised
  • Performance and transfer of data considered

@peterjwest peterjwest changed the base branch from main to prompt-models March 5, 2025 12:02
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