#186 Create new roles module, add D&I Advisor doc (closes #89)
Merged by jflory7. Opened by jflory7.
jflory7/fedora-diversity add/advisor-role  into  master

Download 186.patch

This commit makes the following changes:

  • Antora: Create new roles module for role handbooks
  • roles: Add D&I Advisor role page (closes #89)

This commit creates a new Antora module called roles. This module will
be a place to collect profiles of team responsibilities and how you can
contribute to the team. For now, it will just feature the D&I Advisor
role, but I am excited to see this expand to other things too.

This commit also adds a first draft of the D&I Advisor role to the docs
page. This page was adapted from the existing document in the Council
docs repo:

https://docs.fedoraproject.org/en-US/council/diversity-advisor/

I used this as a base and modified it. I plan to follow up with a Pull
Request to the council-docs repo to remove that page and replace links
with redirects to the D&I Team docs page.

I did make changes though. I used the FESCo representative role page for
many of the additions around duration and the selection process:

https://docs.fedoraproject.org/en-US/fesco/Fedora_Council_Engineering_Rep/

Closes #89. Finally!

Screenshot of local preview of the new role page for the Fedora D&I Advisor

rebased onto c655d3139d8fcd2b7582e423e1605898d21c2c50

Metadata Update from @jflory7:
- Pull-request tagged with: new change, type - docs

It is really nice to have everything related to D&I in one place :100:
+1 from me :grinning:

Looks Good to me, I wonder if there's a way we can mirror the Council Page for D&I Advisor for it, What do you think ?

Thanks for the +1's @ramyaparimi and @nasirhm. :tada: I think this is OK to merge from a team point-of-view. Before merging, I also want to give @tatica, @jonatoni, and @amsharma a chance to weigh in as past D&I Advisors, and for @riecatnor to review from the FCAIC point-of-view.

@nasirhm wrote:
I wonder if there's a way we can mirror the Council Page for D&I Advisor for it, What do you think ?

Yes. I plan to do two things after this PR is merged:

  1. Add a redirect to docs.fp.o site builder for council/diversity-advisor/ to diversity-inclusion/roles/council-advisor/
  2. Delete page in Council docs, change existing xref links in other Council docs pages to the new page location here

I think these two changes will be sufficient to avoid 404s for people with bookmarks or other hard-coded URLs on the Internet. The redirect will also help out with Search Engine Optimization and indexing.

I followed up for feedback in the IRC channel. Going to leave this open until Wednesday, 2 December. If no further feedback by then, I will merge it and work on the other pending actions listed in Fedora-Council/council-docs#96.

@jflory7 wrote:
Yes. I plan to do two things after this PR is merged:

  1. Add a redirect to docs.fp.o site builder for council/diversity-advisor/ to diversity-inclusion/roles/council-advisor/
  2. Delete page in Council docs, change existing xref links in other Council docs pages to the new page location here

I think these two changes will be sufficient to avoid 404s for people with bookmarks or other hard-coded URLs on the Internet. The redirect will also help out with Search Engine Optimization and indexing.

Sounds like an awesome plan @jflory7 , Thank you for working on it :D

It'll be a great move towards improving our documentation by using modules and for the future, once we will have other roles for the D&I Team. It'll be good to have a separate module for roles for better management.

Thanks @nasirhm!

Today is Thursday, 3 December. I am going to merge this Pull Request and follow up on the other pending actions to do a graceful migration from the Council docs to the D&I docs.

Merging. :ocean:

Pull-Request has been merged by jflory7

The docs for Fedora are failing to build with the following error:

error: Page alias cannot reference an existing page: master@diversity-inclusion:roles:council-advisor.adoc

Any chance it's related to this PR?

@dustymabe Thank you for letting us know, I'm taking a look at it.

Metadata