Skip to content

Commit 210f472

Browse files
create planet-intern-instructions.md (fixes #3606) (#3607)
1 parent 37985c0 commit 210f472

File tree

1 file changed

+132
-0
lines changed

1 file changed

+132
-0
lines changed
Lines changed: 132 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,132 @@
1+
# Day-to-Day Guide for Planet Virtual Interns
2+
3+
Now that you’ve completed the 10 steps, you’re ready to become a productive member of the team. As a virtual intern, your 4 core responsibilities are:
4+
5+
- Testing the app
6+
- Creating issues
7+
- Reviewing others’ pull requests
8+
- Submitting your own pull requests
9+
10+
You can always check your day-to-day progress on your GitHub profile, where it’s reflected as points. Our hope is that you find satisfaction watching your contributions climb.
11+
12+
## Scheduling
13+
14+
Your schedule is largely up to you, although we generally require 20–24 hours per week, Monday–Friday between 8AM–6PM EST. This ensures that you’re able to work alongside your team leaders and other interns. This schedule also helps you get up to speed as soon as possible in order to begin making meaningful contributions.
15+
16+
- Post your schedule in the Discord, so we know when we can expect to work with you.
17+
- If there will be permanent changes, post an updated schedule.
18+
- If you cannot make it to a shift, whenever possible, give us 1 work day’s notice.
19+
20+
## Recommended Daily Workflow
21+
22+
Your role offers flexibility in terms of how you schedule your tasks. Over time, you will find a workflow that fits you. One effective way to organize your day-to-day tasks is in the following order:
23+
24+
### [Check into the Virtual Office](https://talk.ole.org/)
25+
26+
As a new intern, it’s a good idea to verbally check in with Mutugi when you arrive. After a week or two, you can just get right to your tasks if you feel comfortable doing so.
27+
28+
### [Review Pull Requests](https://github.com/open-learning-exchange/planet/pulls)
29+
30+
- For each PR, look at the linked issue and test thoroughly to ensure that the PR fixes the issue without creating additional issues and that the code is clean and maintainable.
31+
- If you need to give feedback beyond simply approving the PR, screenshots and screen recordings are always helpful.
32+
33+
### [Find an issue](https://github.com/open-learning-exchange/planet/issues)
34+
35+
- See if there are issues assigned to you, or assign an issue to yourself if there is an unclaimed one that you feel is a good fit for you.
36+
37+
### Begin Working on Your Assigned Issues
38+
39+
- In general, the best practice is to give each issue its own branch so that PRs are easy to understand and merge conflicts are minimized.
40+
41+
### Create a PR Once You Feel the Issue is Solved
42+
43+
Before your very first PR, open the terminal in the app directory and run:
44+
45+
```
46+
npm run install-hooks
47+
```
48+
49+
This prevents [linting issues](https://www.testim.io/blog/what-is-a-linter-heres-a-definition-and-quick-start-guide/) in your pull requests.
50+
51+
To test that your linters are working, run:
52+
53+
```
54+
ng lint
55+
```
56+
57+
If you see general linting errors, try running:
58+
59+
```
60+
ng lint --fix
61+
```
62+
63+
If you see linting errors related to the Chat API, run:
64+
65+
```
66+
cd chatapi
67+
npm install
68+
cd ..
69+
```
70+
71+
Now when you push changes, you should see both tslint for the file and chatapi are passed.
72+
73+
If your PR is a work in progress, feel free to submit it as a draft so you can get back to it later.
74+
75+
#### The Formatting of Our Pull Requests is Very Important
76+
77+
**Title:**
78+
79+
- Use lower case
80+
- Begin with the relevant page and a colon, e.g. `resources:`, `courses:`, `manager:`
81+
- Keep the rest positive and concise, e.g. `smoother date filtering`, `more responsive toolbar`, `cleaner menu formatting`
82+
- End the title with the issue number in parentheses, e.g. `(fixes #1234)`
83+
- Example: `manager: smoother date filtering (fixes #8145)`
84+
85+
You may want to look at [closed PRs](https://github.com/open-learning-exchange/planet/pulls?q=is%3Apr+is%3Aclosed) for more examples.
86+
87+
**Body:**
88+
89+
- Begin the body of the PR with the issue number, e.g. `fixes #1234`, which will helpfully link directly to the issue for those reviewing the PR
90+
- Include a brief description, unless the change is very simple
91+
- Include screenshots and/or screen recordings if you think more information would be helpful
92+
93+
### Check Out
94+
95+
This is one of your most important responsibilities. When it’s nearing the end of your shift, try to verbally check out with Mutugi or Dogi. This is a good chance to show off the progress you made today.
96+
97+
If neither of them are available, it is acceptable to check out in the [Planet Discord channel](https://discord.com/channels/1079980988421132369/1243223477608124426)
98+
.
99+
100+
Example checkout message:
101+
102+
- Number of PRs created
103+
- Number of PRs reviewed
104+
- Number of issues created
105+
- When we will see you next
106+
107+
## Thursday Testing Day
108+
109+
On Thursdays we test the app. Since we spend most of the week testing our production environment, we use this opportunity to test the production server, which you created in [Step 3](https://open-learning-exchange.github.io/#!pages/vi/vi-first-steps.md#Step_3_-_Planet_and_Docker). Make sure it is up to date:
110+
111+
1. Log into your community, go to Manager Settings > Manage Sync
112+
2. Click Run Sync, wait a few moments, and then click Refresh
113+
3. Most, if not all, of the items listed should have a cloud/checkmark icon
114+
115+
A good place to start testing the app is by navigating to the [Tags page on GitHub](https://github.com/open-learning-exchange/planet/tags).
116+
117+
See the Planet Discord channel where Mutugi posts the weekly test versions. Test all the tags between the version jump, and confirm that the changes are working correctly.
118+
119+
Spend some time looking around the app and create issues as you see fit. When in doubt, ask Mutugi if an issue you found is valid.
120+
121+
## Friday Meeting
122+
123+
On Fridays at 11AM EST in the virtual office, the team leaders meet with Richard Rowe, OLE’s president and CEO to discuss our weekly progress and future plans. While interns rarely participate, you are welcome to listen in. We have a very flat management structure at OLE and feel that interns can benefit from the transparency with which the organization is guided. There is usually an opportunity later in the day for you to give your feedback on the meeting, if you have any.
124+
125+
## Virtual Office Etiquette
126+
127+
Our team works best when we treat the virtual office like a physical one. That means being accessible for communication by spending most of your shift logged into the virtual office. We affectionately call it the Planet radio station, so feel free to tune in and out as needed. If the conversation isn’t relevant to you and becomes distracting, go ahead and mute the tab, just don’t forget to turn the volume back up after a little while. Staying connected keeps our team energy flowing.
128+
129+
- Please keep yourself muted when you’re not talking to avoid distracting others.
130+
- If you're having audio or connection issues, try exiting and rejoining.
131+
- If you need to step away, put a message in the chat to let us know.
132+
- Stay in touch! If you ever have questions about any part of your workflow, the virtual office is the perfect place to ask.

0 commit comments

Comments
 (0)