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.

Sign up for GitHub

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

Jump to bottom

Updated Civic Tech Index project profile 2728 #3114

Merged
merged 4 commits into from
May 10, 2022
Merged

Updated Civic Tech Index project profile 2728 #3114

merged 4 commits into from
May 10, 2022

Conversation

Wny-Duong
Copy link
Member

Fixes #2728

What changes did you make and why did you make them ?

  • Updated Current Project Team roster
    • Removed: Levi Adissi, Iffath Sultana, Maria Studnicka, Shipra Prakash, Emily Marinaro)
    • Added: Sean Sada, Dennis Chatkhan, Briana Bennett, Joyce Kim, Adaeze Nwaezeigwe, Ron Fu, Vance Dubberly, Isaiah Ozadhe
    • Note: Nandana Rao was listed as a new addition, but had been added to the roster 10 months ago.
  • Added to technologies and tools as they were written in the comments under the original issue.
    • Tools: Google docs, Figma, Usability Hub
    • Technologies: AWS
  • Updated links to Github and wiki pages

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied

image
image

Visuals after changes are applied

image
image

@github-actions
Copy link

github-actions bot commented May 9, 2022

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.

git checkout -b Wny-Duong-update-project-file-2728 gh-pages
git pull https://github.com/Wny-Duong/website.git update-project-file-2728

@github-actions github-actions bot added P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less Complexity: Small Take this type of issues after the successful merge of your second good first issue time sensitive Needs to be worked on by a particular timeframe labels May 9, 2022
@harshitasao harshitasao self-requested a review May 9, 2022 09:42
@harshitasao
Copy link
Member

Availability: 6pm-7pm IST
Review ETA: End of the day 09/05/2022

@Sparky-code Sparky-code self-requested a review May 9, 2022 20:33
harshitasao
harshitasao approved these changes May 10, 2022
View reviewed changes
Copy link
Member

@harshitasao harshitasao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Wny-Duong Good work! The changes look good to me.

@Sparky-code
Copy link
Member

Avail: 1hr
ETA: 5/10 EOD

Sparky-code
Sparky-code approved these changes May 10, 2022
View reviewed changes
Copy link
Member

@Sparky-code Sparky-code left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Wny-Duong Looks great! All the links look right and the tools and technologies you added match the referenced issue.

@Sparky-code Sparky-code merged commit ac243ea into hackforla:gh-pages May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sparky-code Sparky-code Sparky-code approved these changes

@harshitasao harshitasao harshitasao approved these changes

Assignees
No one assigned
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less time sensitive Needs to be worked on by a particular timeframe
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Project Profile: Civic Tech Index
3 participants
@Wny-Duong @harshitasao @Sparky-code