council/docs

Fork 5

Update council legal page #167

Merged

jflory7 merged 3 commits from jl-council-legal-page into main

2026-06-09 01:55:43 +00:00

jlovejoy commented

2023-01-05 23:27:05 +00:00

Contributor

Add link to Fedora-legal docs section
remove export policy and add link to correct/updated one on Fedora-legal docs
rename page
rename link to page in Nav

Signed-off-by: Jilayne Lovejoy

Add link to Fedora-legal docs section remove export policy and add link to correct/updated one on Fedora-legal docs rename page rename link to page in Nav Signed-off-by: Jilayne Lovejoy

jlovejoy commented

2023-01-05 23:27:25 +00:00

Author

Contributor

fixed #164

bcotton commented

2023-01-06 13:37:36 +00:00

Contributor

Instead of a bare URL, let's put in an xref (e.g. ..please see the xref:legal::index.adoc[legal documentation])

Instead of a bare URL, let's put in an xref (e.g. `..please see the xref:legal::index.adoc[legal documentation]`)

bcotton commented

2023-01-06 13:38:55 +00:00

Contributor

Metadata Update from @bcotton:

Request assigned

**Metadata Update from @bcotton**: - Request assigned

bcotton commented

2023-01-06 13:38:57 +00:00

Contributor

Looks good to me. I can fix the URL/xref issue I mentioned above post-merge.

I'll leave this open until early next week in case another Council member has comments.

Looks good to me. I can fix the [URL/xref issue](https://pagure.io/Fedora-Council/council-docs/pull-request/167#comment-181539) I mentioned above post-merge. I'll leave this open until early next week in case another Council member has comments.

jflory7 commented

2023-01-06 14:24:10 +00:00

Owner

+1 with xref

+1 with `xref`

jflory7 commented

2023-01-06 14:38:07 +00:00

Owner

Changing my vote to a weak -1 – I'd prefer that we delete the page in Council docs, update the navigation sidebar to link to the new page, and use :page-aliases: to redirect traffic at the old URL to the new one in the Legal docs.

Changing my vote to a weak `-1` – I'd prefer that we delete the page in Council docs, update the navigation sidebar to link to the new page, and use `:page-aliases:` to redirect traffic at the old URL to the new one in the Legal docs.

dcantrell commented

2023-01-06 16:26:48 +00:00

Contributor

I agree with @jflory7. Might as well delete the page and do the redirect.

bcotton commented

2023-01-10 21:45:32 +00:00

Contributor

The main argument for keeping the page is if we want to contextualize the link to the Legal docs. But we can add that later if there's context we want to add.

I'm going to go ahead and merge this and then immediately follow up with a deletion and a PR to the legal docs to add the alias.

The main argument for keeping the page is if we want to contextualize the link to the Legal docs. But we can add that later if there's context we want to add. I'm going to go ahead and merge this and then immediately follow up with a deletion and a PR to the legal docs to add the alias.

bcotton commented

2023-01-10 21:45:42 +00:00

Contributor

Pull-Request has been merged by bcotton