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

Andrew s/slimmer even more #53

Merged
merged 12 commits into from
May 29, 2024
Merged

Conversation

andrew-sotiriou
Copy link

@andrew-sotiriou andrew-sotiriou commented May 16, 2024

What does this PR do?

Removing more things

How can this change be manually tested?

....

Asana Card

https://app.asana.com/0/1198498632694766/1207337084848986/f

Questions or concerns about this change?

n/a

Deployment Notes

Will be tied to a Oxygen PR:


PR Readiness

  • Changes have been validated in local dev

Code Readiness

Automated Testing

  • Unit tests are written at the appropriate level given the code’s risk-level
  • Existing tests still pass
  • No unnecessary test runner output, e.g. debugging or warnings are not present in Jest or Unittest output

Other Considerations

  • Relevant documentation updated in Slite

Also consider this PR's impact on architecture and security; AMP, Apple News, RSS; SEO, GA, performance and accessibility.

Please see Post Merge for additional responsibilities after this PR is merged.

@andrew-sotiriou andrew-sotiriou merged commit f8a0549 into master May 29, 2024
@andrew-sotiriou andrew-sotiriou mentioned this pull request May 29, 2024
@andrew-sotiriou andrew-sotiriou deleted the andrew-s/slimmer-even-more branch May 29, 2024 14:59
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