-
-
Notifications
You must be signed in to change notification settings - Fork 785
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
Removed mention of CfA from the Join Us page #5645
Removed mention of CfA from the Join Us page #5645
Conversation
Want to review this pull request? Take a look at this documentation for a step by step guide! From your project repository, check out a new branch and test the changes.
|
ETA: 10/4/23 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@YolandaHaynes Good job: The branches are correct and the issue is linked. The changes are visible on my system on desktop, mobile and tablet. The correct files were changed and the links lead to the correct stripe donation page. I also see that the you changed the name for styling from "gif" to "png" since the image is now in a different format from the original. Well done!
ETA: 10/03/23 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good on my end with just some small accessibility nitpicks
Availability: 12pm - 6pm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The following are confirmed:
- Correct branch and linked issue
- Replaced code with the updated version
- Replaced the donation form graphic and is linked correctly
- Changes are shown in different view modes
- Confirmed before/after screenshots
You did an outstanding job @YolandaHaynes! Thanks for contributing.
As for making further changes than those listed, I'd recommend to follow the instructions and images as given.
If there are any questions or concerns, it's best to consult with the #hfla-site slack channel or a team lead. There are also more instructions on your Pre-work Checklist
Hi @jaasonw & @YolandaHaynes. The accessibility findings are good points to bring up, but they are not within the scope of the issue and can be addressed in future issues. We encourage devs to follow the action items in the original issue as closely as possible. As @agutiernc brought up, any findings can be brought up on the slack channel or directly with the merge team. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM then 🚀
Fixes #5447
What changes did you make?
Why did you make the changes (we will use this info to test)?
Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)
Visuals before changes are applied
Visuals after changes are applied