hackforla / website

Hack for LA's website
https://www.hackforla.org
GNU General Public License v2.0
325 stars 778 forks source link

Skills Issue: Developer: Kristen Cardon #7326

Open k-cardon opened 3 months ago

k-cardon commented 3 months ago

Prerequisite

We are looking forward to having you on our team. Please make sure to attend the general Hack for LA onboarding to get the process started https://meetup.com/hackforla/events.

Overview

As a developer on the Website team this issue will be your companion and a place to track your progress with the path we have set out for you.

Special Notes

  1. This issue will stay open for as long as you are on the Website team. Use it as a place to indicate that you have completed a level as well as get instructions on how to progress.
  2. Usually we don't want you to have more than one issue assigned to you at a time, this issue is the exception, because it is instructions on how to work on other issues. Do not close this issue until you leave the team (please see to do items associated with leaving professionally).
  3. The action items listed below should mostly be worked on in a sequential order. However, you don't have to wait on one if you can proceed with the others. For instance, you don't have to wait for attending a weekly meeting before setting up your dev environment.
  4. During the general Hack for LA onboarding, you will be directed to fill out a form that will add you to the Website team Google Drive and GitHub teams, and then you will add yourself to the roster. If you have not done that yet, you will not be able to do the action items in section 1.
  5. The template that this issue is made from is a work in progress. We will be updating it, and possibly updating your issue. It works through section 17. But after that it's still a work in progress. If any of the links don't work, please leave a note in the comments on this issue https://github.com/hackforla/website/issues/4944, and we will get you an update.

Action Items

Table of Contents

Sections 1 - Joining the website team 2 - Team Meetings (Options and Requirements) 3 - Development Environment Setup 4 - First GitHub Issue (GFI) 5 - Weekly Updates 6 - 1st Pull Request 7 - Additional reading 1 8 - 2nd good first issue 9 - Pull Request Reviews - GFI 10 - Additional reading 2 11 - Small Issue 12 - Pull Request Reviews - Small 13 - Issue Making - Level 1 (GFI & Small) 14 - Medium Issue 14.1 Issue Making - Level 2 (Medium) 15 - Pull Request Reviews - Medium 16 - Issue Making - Level 2 (GFI) 17 - Merge Team Skills Review FAQ Resources

1 - JOINING THE WEBSITE TEAM.

Table of Contents

2 - TEAM MEETINGS (OPTIONS AND REQUIREMENTS)

Table of Contents

3 - DEVELOPMENT ENVIRONMENT SETUP

Table of Contents

4 - FINDING AND ASSIGNING YOUR FIRST GITHUB ISSUE (GFI)

Table of Contents

5 - GIVING WEEKLY UPDATES ON YOUR DEVELOPMENT ISSUES

Table of Contents

6 - SUBMITTING YOUR FIRST PULL REQUEST

Table of Contents

7 - ADVANCED READING TO READY YOU FOR LARGER MORE COMPLEX ISSUES

Table of Contents

8 - MOVE ON TO 2ND GOOD FIRST ISSUE (AKA, IT GETS EASIER AND DID YOU BRANCH CORRECTLY?)

Table of Contents

9 - GOOD FIRST ISSUE (GFI) PULL REQUEST REVIEWS

Now that you have two merged good first issue PRs, you are eligible to review good first issue PRs, Review Required from other people who are following in the same journey path as you.

See How to review Pull Requests guide will teach you how to review pull requests.

Please review 5 good first issue PRs. Each PR requires at least two reviews, so by reviewing 5 good first issue PRs you are repaying the effort that others did for you (provided 4 reviews for your 2 good first issues) plus 1 extra review to help us all make up the deficit for people who submit a PR but don't get this far.

Table of Contents

10 - UNDERSTAND HOW TO PROGRESS THROUGH ISSUES IN THE PRIORITIZED BACKLOG AND ON ISSUE MAKING AND TEMPERATURE CHECK

Congrats on making it this far. Issues get more complicated from here, either they include more changes, or have several files to change or you have to research something that we are unsure how to do, or there is complicated logic that needs writing or rewriting. Each issue size that you take on will guide you to a more complicated level in sequence, and you can see from the labels and overviews what they are about.

Its important that you try to work on issues that fill in gaps in your knowledge (see the self tests for a reminder about what to look for).

So keep going, the fun stuff is about to start.

Having said that, we are also going to have you take on some issue making (surprise! There is no issue making fairy, only volunteers like you that created issues for the people that come after them). Pay attention to how the issues you have already worked on are constructed and how they change as they go up the ladder. That way when we start you on the issue making portion of the team work, you will know what you are shooting for when its your time to make issues.

Also, we want you on the Merge team. This will ensure you are a competent developer and an awesome collaborative contributor to any team you join in the future.

[THIS WHOLE THING COULD BE MOVED TO A WIKI PAGE THAT EXPLAINS THE VALUE TO THEIR CAREER AND HAVE A TLDR HERE]

Table of Contents

11 - MOVING ON TO A SMALL ISSUE

Table of Contents

12 - PULL REQUEST REVIEWS - SMALL

Now that you have your small PR merged, you are eligible to review small PRs, Review Required from other people who are following in the same journey path as you.

Please review 3 small PRs. Each PR requires at least two reviews, so by reviewing 3 small PRs you are repaying the effort that others did for you (provided 2 reviews for your 1 small issue PR) plus 1 extra review to help us all make up the deficit for people who submit small PRs and then drop off the team.

Table of Contents

13 - GET EXPERIENCE MAKING ISSUES - LEVEL 1 (GFI & Small)

Creating issues from templates will give you experience on how issues

and like the good first and small issues you have already done, they are perscritive enough to do with no prior experience issue making.

Table of Contents

14 - MOVING ON TO A MEDIUM ISSUE

Table of Contents

14.1 MAKE A MEDIUM ISSUE FROM AN ER OR EPIC

Only work on this section if you needed a medium issue and one is not available from the prioritized backlog

Table of Contents

15 - PULL REQUEST REVIEWS - Medium

Now that you have your medium PR merged, you are eligible to review medium PRs, Review Required from other people who are following in the same journey path as you.

Please review 3 medium PRs. Each PR requires at least two reviews, so by reviewing 3 medium PRs you are repaying the effort that others did for you (provided 2 reviews for your 1 medium issue PR) plus 1 extra review to help us all make up the deficit for people who submit medium PRs and then drop off the team.

Table of Contents

16 - ISSUE MAKING - LEVEL 2, GFI

Table of Contents

17 - MERGE TEAM SKILLS REVIEW

We want everyone who joins this team to get onto the merge team so that you can get experience running meetings and office hours, mentoring, creating sufficent workflow for the team, escalations, and ultimately being responsible for final approval and merging of pull requests made by team members on lower sections. At this point we will check to see if you are ready to join the merge team, or what your next steps are to get you closer to ready.

Table of Contents

FAQ section

Are there exceptions to which size issues I work on? - Medium (you can work on one medium issue, but only one at a timeone per person, with some exceptions, see below) - Large (you can work on more than one large issue, but only one at a time) - The reasons for this progression are: - The issues start out as being prescriptive and become less so as you gain more experience by working through increasingly complex issues. - We are trying to teach you the team methodology through the issues themselves. - It ensures you understand what we expect and the quality of contributions. - You can work on back-to-back small issues if it meets the following criteria: - You are learning something new and need to work on an issue of a lesser complexity - Since we have a limited number of these, you must get approval from lead or pm - You can work on a second medium issue if it meets the following criteria: - You are learning something new and need to work on an issue of a lesser complexity - Since we have a limited number of these, you must get approval from lead or pm
What should I do if I have a question about an issue I'm working on, and I haven't gotten a response yet? - First, you should post the question or blocker as a comment on your assigned issue, so it can be easily referred to in the next bullet points. - Then, add the label `Status: Help Wanted` so other developers can see it and potentially help answer your question. In addition, you will still need to post a Slack message or bring it up in meeting so we know you need help; see below for how to do that. - Also, you can post your question on the hfla-site Slack channel and link the issue you're working on, so other developers can see and respond. - Lastly, you can add the issue to the "Development team meeting discussion items" "Questions/In Review" column of the Project Board so that it can be addressed in the next development meeting. Please bring it during the meeting that you need help.
If you need to take some time off from the team Copy and customize this response, and leave it in a comment on this issue ``` I need to take some time off from the team. I believe I will be back on [Replace with DATE YOU WILL BE BACK] ``` - If you have put work in, please make sure its well documented. - Add the label `away on hold` - Move this issue to the questions column - if you are assigned to an issue, if you have not yet made any progress on the issue, please unassign.

Table of Contents

Resources/Instructions

Table of Contents


Skills List - INTAKE

Skills List, self test on Intake, fill out when you join the team, don't update

Front End

Return to section 1 section 10

Skills List - ONGOING

Skills List, update as you do work on this team

Front End

Table of Contents

github-actions[bot] commented 3 months ago

Hi @k-cardon.

Please don't forget to add the proper labels to this issue. Currently, the labels for the following are missing:

NOTE: Please ignore this comment if you do not have 'write' access to this directory.

To add a label, take a look at Github's documentation here.

Also, don't forget to remove the "missing labels" afterwards. To remove a label, the process is similar to adding a label, but you select a currently added label to remove it.

After the proper labels are added, the merge team will review the issue and add a "Ready for Prioritization" label once it is ready for prioritization.

Additional Resources:

k-cardon commented 3 months ago

1 - JOINING THE WEBSITE TEAM update

How many hours did it take you to finish this step?

A: 0 hours, 10 minutes

return to section 1

k-cardon commented 3 months ago

2 - TEAM MEETINGS update

which meetings did you register for

  • [x ] Developer (front-end/back-end)
  • [x] (Optional) Office Hours
  • [x ] All team meeting When did you attend your first team meeting?

A: 8/25/24

return to section 2

k-cardon commented 3 months ago

3 - GETTING YOUR DEVELOPMENT ENVIRONMENT SETUP update

How many hours did it take you to finish this step?

A: 0 hrs 15 min

return to section 3

k-cardon commented 3 months ago

4 - FINDING AND ASSIGNING YOUR FIRST GITHUB ISSUE update

How many hours did it take you to finish this step?

A: 0 hrs 10 min

return to section 4

k-cardon commented 3 months ago

6 - PULL REQUESTS update

What is the number of your first merged pull request?

  • 7348

Did you receive any reviews that required you to change anything on your PR?

  • [x ] no
  • [ ] yes (if yes, describe what you learned)

Comments:

return to section 6

k-cardon commented 3 months ago

7 - ADVANCED READING TO READY YOU FOR LARGER MORE COMPLEX ISSUES update

How many hours did it take you to finish this step?

A: 0 hrs 30 min

Do you have any questions about what you read?

  • [ ] yes, I had questions, and I left comments in the appropriate issues [WE NEED TO UPDATE THOSE TWO RESOURCES TO HAVE LINKS TO ISSUES WHERE PEOPLE CAN PUT QUESTIONS AND MOVE THE ISSUES TO THE QUESTIONS/REVIEW COLUMN]
  • [x ] no, I did not have any questions

return to section 7

k-cardon commented 3 months ago

9 - PULL REQUEST REVIEWS - GFI - Update

I have reviewed a good first issue PR #7350

Did you catch anything?

return to section 9

k-cardon commented 3 months ago

9 - PULL REQUEST REVIEWS - GFI - Update

I have reviewed a good first issue PR #7365

Did you catch anything?

return to section 9

k-cardon commented 3 months ago

9 - PULL REQUEST REVIEWS - GFI - Update

There are currently no good first issue PRs to review, but ill check back later.

return to section 9

k-cardon commented 3 months ago

10 - UNDERSTAND HOW TO PROGRESS THROUGH ISSUES IN THE PRIORITIZED BACKLOG AND ON ISSUE MAKING update

Up to now we have just been getting you ready. Now the fun starts. Are you continuing?

  • [x] I'm so ready, bring it on (continuing)
  • [ ] I am worn out from the setup and the good first issues but still game (continuing)
  • [ ] I won't be continuing, (please let us know why and close this issue)

Comments:

return to section 10

k-cardon commented 3 months ago

8 - MOVE ON TO 2ND GOOD FIRST ISSUE update

What is the number of your 2nd merged pull request?

  • 7364

    Did you receive any reviews that required you to change anything on your PR?

  • [x] no
  • [ ] yes (if yes, describe what you learned)

Comments:

return to section 8

k-cardon commented 2 months ago

5 - GIVING WEEKLY UPDATES ON YOUR DEVELOPMENT ISSUES update

on what issue did you give your first weekly update?

return to section 5

k-cardon commented 2 months ago

9 - PULL REQUEST REVIEWS - GFI - Update

I have reviewed a good first issue PR #

Did you catch anything?

return to section 9

k-cardon commented 2 months ago

9 - PULL REQUEST REVIEWS - GFI - Update

I have reviewed a good first issue PR #7370

Did you catch anything?

return to section 9

k-cardon commented 2 months ago

9 - PULL REQUEST REVIEWS - GFI - Update

I have reviewed a good first issue PR #7371

Did you catch anything?

return to section 9

k-cardon commented 2 months ago

13 - GET EXPERIENCE MAKING ISSUES - LEVEL 1 (GFI & Small) update

Which EPIC or ER did you work on (provide the issue number)

7165

How many hours did it take you to make the issue(s)?

Number of hours: 0 hrs 30 min

Did you find anything required clarification or anything we could improve about the instructions?

Suggestions for improvement: no, it was clear, thank you! (the only minor issue was that one of the lines on the spreadsheet assigned to me had already been completed by an issue last year)

return to section 13

HackforLABot commented 2 months ago

@k-cardon

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures (optional): "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the Questions/In Review column of the Project Board and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel. Please note that including your questions in the issue comments- along with screenshots, if applicable- will help us to help you. Here and here are examples of well-formed questions.

You are receiving this comment because your last comment was before Tuesday, September 10, 2024 at 12:04 AM PST.

k-cardon commented 2 months ago
Progress: Initial stages
Blockers: Was delayed starting, have some family emergencies this week
Availability: A few hours 
ETA: End of week
Pictures (optional): N/A
k-cardon commented 2 months ago

11 - SMALL update

What is the number of your small merged pull request?

  • 7386

    Did you receive any reviews that required you to change anything on your PR?

  • [x] no
  • [ ] yes (if yes, describe what you learned)

Comments: no PR; the issue was to note any spell check errors in a file

return to section 11

k-cardon commented 2 months ago

12 - PULL REQUEST REVIEWS - Small - Update

I have reviewed a small PR #7466

Did you catch anything?

return to section 12

HackforLABot commented 2 months ago

Hello @k-cardon, we appreciate you taking on issue #7457, however it looks like you've already worked on enough issues of this complexity. Try checking out some of the issues of the next complexity from the Prioritized Backlog :)

We are going to unassign you from this issue so you can take on another issue.

Hfla appreciates you! :)

P.S. There is one exception to this rule/automation, and that is if you were away for a long time, and need to do the issue ladder again. If that is the case, please post the following note on the issue and on your Skills Issue (Pre-work Checklist). A Merge team member will reassign you to this issue, and will help you get assigned to subsequent issues up to medium size.

I am returning after a significant time away, and need to do the issue ladder again. Please assign me back to this issue.
HackforLABot commented 2 months ago

Hello @k-cardon, we appreciate you taking on issue #7456, however it looks like you've already worked on enough issues of this complexity. Try checking out some of the issues of the next complexity from the Prioritized Backlog :)

We are going to unassign you from this issue so you can take on another issue.

Hfla appreciates you! :)

P.S. There is one exception to this rule/automation, and that is if you were away for a long time, and need to do the issue ladder again. If that is the case, please post the following note on the issue and on your Skills Issue (Pre-work Checklist). A Merge team member will reassign you to this issue, and will help you get assigned to subsequent issues up to medium size.

I am returning after a significant time away, and need to do the issue ladder again. Please assign me back to this issue.
k-cardon commented 2 months ago

12 - PULL REQUEST REVIEWS - Small - Update

There are currently no small PRs to review, but i'll check back later.

return to section 12

k-cardon commented 2 months ago

12 - PULL REQUEST REVIEWS - Small - Update

I have reviewed a small PR #7496

Did you catch anything?

return to section 12

k-cardon commented 2 months ago

12 - PULL REQUEST REVIEWS - Small - Update

I have reviewed a small PR #7522

Did you catch anything?

return to section 12

k-cardon commented 2 months ago

14 - MEDIUM update

What is the number of your medium merged pull request?

  • 3290

    Did you receive any reviews that required you to change anything on your PR?

  • [ ] no
  • [ ] yes (if yes, describe what you learned)

Comments:

return to section 14

HackforLABot commented 1 month ago

@k-cardon

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures (optional): "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the Questions/In Review column of the Project Board and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel. Please note that including your questions in the issue comments- along with screenshots, if applicable- will help us to help you. Here and here are examples of well-formed questions.

You are receiving this comment because your last comment was before Tuesday, October 1, 2024 at 12:04 AM PST.

k-cardon commented 1 month ago
Progress: Completed issue #3290 but need to add a decision record
Blockers: Time constraints
Availability: Should have a few hours available this week
ETA: End of week
k-cardon commented 1 month ago

14 - MEDIUM update

What is the number of your medium merged pull request?

  • 3290

    Did you receive any reviews that required you to change anything on your PR?

  • [x] no
  • [ ] yes (if yes, describe what you learned)

Comments: There was no PR for this issue; I had to add documentation for the migrating process

return to section 14

k-cardon commented 1 month ago

15 - PULL REQUEST REVIEWS - Medium - Update

I have reviewed a medium PR #7569

Did you catch anything?

return to section 15

k-cardon commented 1 month ago

15 - PULL REQUEST REVIEWS - Medium - Update

I have reviewed a medium PR #7589

Did you catch anything?

return to section 15

k-cardon commented 1 month ago

15 - PULL REQUEST REVIEWS - Medium - Update

There are currently no medium PRs to review, but i'll check back later.

return to section 15

k-cardon commented 1 month ago

I'm unable to progress at the moment--there are no medium PRs to review, there are no epics/ERs ready to be turned into issues (level 2, GFI), and when I went to the office hours last week to discuss no one from leadership was there to answer questions--as a result, I am currently blocked.

k-cardon commented 1 month ago

15 - PULL REQUEST REVIEWS - Medium - Update

I have reviewed a medium PR #7616

Did you catch anything?

return to section 15

k-cardon commented 1 month ago

I have finished sections 1-16 and am ready to have my activity reviewed by the merge team

return to section 17

HackforLABot commented 3 weeks ago

@k-cardon

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures (optional): "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the Questions/In Review column of the Project Board and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel. Please note that including your questions in the issue comments- along with screenshots, if applicable- will help us to help you. Here and here are examples of well-formed questions.

You are receiving this comment because your last comment was before Monday, November 4, 2024 at 11:04 PM PST.

k-cardon commented 3 weeks ago

Progress: Waiting for merge team review Blockers: Haven't had response from merge team to slack message Availability: n/a, no issue assigned ETA: n/a

k-cardon commented 2 weeks ago

I started working on the Home United Us project this week and would like to transition there from the website team; will join the office hours briefly today to discuss any necessary logistics.

ExperimentsInHonesty commented 2 weeks ago

@k-cardon ok see you then.

k-cardon commented 2 weeks ago

@ExperimentsInHonesty could you clarify what more I need to do in order to make this transition? I was waiting for merge team to review my activity and heard back from Roslyn earlier today; she mentioned there might be more I need to conclude on the website before transitioning to HUU, let me know if I should meet with you or with Roslyn for more details.

HackforLABot commented 1 week ago

@k-cardon

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures (optional): "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the Questions/In Review column of the Project Board and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel. Please note that including your questions in the issue comments- along with screenshots, if applicable- will help us to help you. Here and here are examples of well-formed questions.

You are receiving this comment because your last comment was before Monday, November 18, 2024 at 11:04 PM PST.

k-cardon commented 1 week ago

Progress: Haven't had time to work on this project yet Blockers: Very limited time available this week Availability: Very little; will perhaps need to complete this next week ETA: Next Wednesday