issues
search
specollective
/
spec-frontend
Frontend for SPEC's informational website.
3
stars
0
forks
source link
Add contentful integration
#156
Closed
jtorreggiani
closed
1 year ago
jtorreggiani
commented
1 year ago
Pull Request Process
Describe the issue you are trying to resolve
Adding support for dynamic content.
Describe the changes you've made to resolve the issue
Install content client
Refactor code to use dynamic properties.
Add screenshots of the UI before and after your changes have been made
Looks the same.
Code Review Checklist
[X] Have the package.json or package-lock.json been changed? If so, does the pull request description say why?
[X] Has the UI for mobile and desktop been changed, did you test on iPhone and desktop views?
[X] Does the UI match the requirements in the ticket and designs in Figma; does it pass the visual smell test?
[X] Have TypeScript files been changed, does the TypeScript pass the smell test?
[X] If you don't feel super confident in your review, did you assign someone more senior to double-check?
Pull Request Process
Describe the issue you are trying to resolve
Adding support for dynamic content.
Describe the changes you've made to resolve the issue
Add screenshots of the UI before and after your changes have been made
Looks the same.
Code Review Checklist