fix: update Civic Tech Structure org title#8622
fix: update Civic Tech Structure org title#8622nathanjkim-codes wants to merge 1 commit intohackforla:gh-pagesfrom
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. |
jjam02
left a comment
There was a problem hiding this comment.
The code itself looks great so good job on that but there a couple changes that need to be made before I can approve this.
Changes Requested
Changes to be made in pull request
- images do not load in the pull request comment
- if this is a gh issue and not something when you made the comment leave as is
- Check off CodeQL Alerts checkbox
Changes to be made in original Issue
- branch name should have the issue number
- not sure if this can be fixed without creating another pull request
- mark the items as completed when you complete them
- change the project board status from "Prioritized Backlog" to "In Progress (actively working)"
jjam02
left a comment
There was a problem hiding this comment.
Review ETA: 5:30 PM PST 4/30/26
Availability: 12pm-9pm PST Everyday
|
Thank you for the feedback! I updated the PR description and added the screenshots. I also checked the CodeQL Alerts checkbox. I marked the completed action items in the original issue and updated the project board status to “In Progress (actively working).” For the branch name, I’ll make sure to include the issue number in my branch names for future pull requests. Please let me know if anything else needs to be updated. |
|
Review ETA: 8PM PST 5/1/26 |
Fixes #8498
What changes did you make?
Why did you make the changes (we will use this info to test)?
CodeQL Alerts
After the PR has been submitted and the resulting GitHub actions/checks have been completed, developers should check the PR for CodeQL alert annotations.
Check the PR's comments. If present on your PR, the CodeQL alert looks similar as shown
Please let us know that you have checked for CodeQL alerts. Please do not dismiss alerts.
Instructions for resolving CodeQL alerts
If CodeQL alert/annotations appear, refer to How to Resolve CodeQL alerts.
In general, CodeQL alerts should be resolved prior to PR reviews and merging
Screenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes)
Visuals before changes are applied
Visuals after changes are applied