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.

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

gone through team leader onboarding #241

Merged
merged 4 commits into from
Sep 10, 2023
Merged

Conversation

bast
Copy link
Member

@bast bast commented Sep 10, 2023

typo fixes, slight shortening, reorganize where I thought it was helpful

typo fixes, slight shortening, reorganize where I thought
it was helpful
@bast
Copy link
Member Author

bast commented Sep 10, 2023

@samumantha

@samumantha
Copy link
Contributor

👍

The thought about separating tips and tricks and background section was to separate it as "reading material", ie not to go through it during the onboarding.

Would you go through all of this page during onboarding?

@bast
Copy link
Member Author

bast commented Sep 10, 2023

During on-boarding I would skip few sections. We can also put the "background" back if that helps.

@samumantha
Copy link
Contributor

Two comments/questions:

  • The path ahead section is gone now? I thought it was nice to mention this for interested team leaders as we don't really have an official debriefing (where it would fit better?) Having gone that way, I found it helpful to know that there is different ways to get involved and that the project is open to it.

  • the "please do not" section: if you want to have it as part of the onboarding I would get back the short version of this point from before my PR, not this very verbose version.

@samumantha
Copy link
Contributor

During on-boarding I would skip few sections. We can also put the "background" back if that helps.

We could put the sections that are skipped in bottom?

I was trying to find a better place for the background section, but didn't find a good one. Should maybe not be here at all?

Maybe there could be it's own background section in the manuals, where all the info on how CR workshops were done at certain point in time and background info on decisions made is presented?

@bast
Copy link
Member Author

bast commented Sep 10, 2023

OK I will:

  • bring back the path section
  • try to shorten the "what we try to avoid"
  • reorder with most important on top, what can be skipped on bottom

Later we can move the "background" stuff to some other pages but I am now out of time :-)

@samumantha
Copy link
Contributor

Looks good and ready to merge

@bast bast merged commit 63ecce3 into master Sep 10, 2023
@bast bast deleted the radovan/team-leader-onboarding branch September 10, 2023 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants