Skip to content

fix: LADI-5199 Update PR template - phase 2#930

Open
jendiamond wants to merge 1 commit into
mainfrom
LADI-5199_update-pr-template-3
Open

fix: LADI-5199 Update PR template - phase 2#930
jendiamond wants to merge 1 commit into
mainfrom
LADI-5199_update-pr-template-3

Conversation

@jendiamond

@jendiamond jendiamond commented Jun 12, 2026

Copy link
Copy Markdown
Member

Connected to LADI-

Deployed on https://deploy-preview-xxx--uclalibrary-test.netlify.app/


Notes

{add notes about your PR}

Screenshot

{add a screenshot if applicable}


Checklist

Author

  • Browsers
    • Review PR in Chrome, Firefox, Safari, Edge
    • Review PR in desktop view, tablet view, mobile view
  • A11Y
    • Zoom the site to 200%
    • Check for keyboard traps
  • Design & Code
    • Check that it looks like the design(s) (if applicable)
    • Include a working / updated spec file (if applicable)
    • Review the Chromatic (if the PR is large and requires it)

UX Review

  • UX has reviewed and approved

Dev Review

  • Review Chromatic
  • Review the deploy preview
  • Review the code

@jendiamond jendiamond self-assigned this Jun 12, 2026
@jendiamond jendiamond added the fix label Jun 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Chromatic Visual Tests (Cypress)

To conserve our visual snapshot budget, Chromatic is not configured to run automatically. When the PR is ready and other checks are passing, run it manually:

  1. Visit the workflow: https://github.com/UCLALibrary/library-website-nuxt/actions/workflows/chromatic.yml 2. Click Run workflow 3. Select the correct branch for this PR and click Run workflow again

@github-actions

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant