Skip to content

Update technologies and tools fields on the "Access the Data" program page#5121

Merged
t-will-gillis merged 1 commit intohackforla:gh-pagesfrom
davidwiese:update-project-file-access-the-data-4818
Aug 6, 2023
Merged

Update technologies and tools fields on the "Access the Data" program page#5121
t-will-gillis merged 1 commit intohackforla:gh-pagesfrom
davidwiese:update-project-file-access-the-data-4818

Conversation

@davidwiese
Copy link
Member

Fixes #4818

What changes did you make?

  • Updated the tools and technologies fields in access-the-data.md

Why did you make the changes (we will use this info to test)?

  • Updated the project information for Access the Data so that visitors to the website can find accurate and up-to-date information about the tools and technologies currently used.

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

Visuals before changes are applied

image

Visuals after changes are applied

image

@github-actions
Copy link

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 davidwiese-update-project-file-access-the-data-4818 gh-pages
git pull https://github.com/davidwiese/website.git update-project-file-access-the-data-4818

@github-actions github-actions bot added good first issue Good for newcomers role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) time sensitive Needs to be worked on by a particular timeframe ready for dev lead Issues that tech leads or merge team members need to follow up on size: 0.25pt Can be done in 0.5 to 1.5 hours labels Jul 31, 2023
@DorianDeptuch DorianDeptuch self-requested a review August 1, 2023 05:46
@DorianDeptuch
Copy link
Member

Availability: 7/31
ETA: 7/31 evening

Copy link
Member

@DorianDeptuch DorianDeptuch left a comment

Choose a reason for hiding this comment

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

Hello David,

You documented the changes correctly, the code looks as the issue suggests, and you provided photos for the visual updates. I can confirm that the site works and has the correct information on it when it is run in my local environment. Great work!

@roslynwythe roslynwythe self-requested a review August 2, 2023 22:59
@t-will-gillis t-will-gillis self-requested a review August 6, 2023 15:16
Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

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

Hey @davidwiese Great job! Your branches are correct, you are linked back to the issue, your description explains the whats and the why, and you are including screenshots of before and after. Docker confirms your changes.

Thanks for taking up this issue!

@t-will-gillis t-will-gillis merged commit 137bbf6 into hackforla:gh-pages Aug 6, 2023
@davidwiese davidwiese deleted the update-project-file-access-the-data-4818 branch August 6, 2023 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) ready for dev lead Issues that tech leads or merge team members need to follow up on role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours time sensitive Needs to be worked on by a particular timeframe

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Project Profile: Access the Data

3 participants