CE #2674
Merged
CE #2674
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lurch
reviewed
Oct 28, 2022
| "title": "PIP", | ||
| "description": "The Product Information Portal (PIP) for Raspberry Pi compliance documents", | ||
| "title": "Product Information Portal", | ||
| "description": "Raspberry Pi compliance documents", |
There was a problem hiding this comment.
Isn't the intention for PIP to (eventually) be more than just compliance documents?
There was a problem hiding this comment.
Yes. Eventually, most of what's on datasheets.raspberrypi.com will but put into PIP. But not before it's overhauled to get human-readable URLs and allow us to use human-readable document names. A string of numbers and letters which you have to go and look up isn't helpful. Documents should have human-readable names.
There was a problem hiding this comment.
The names are automatically generated by Cognidox. Not sure there is a way round that.
There was a problem hiding this comment.
Sure, but PIP isn't raw Cognidox. Paul and Chris are going to look at it.
|
It's already used for more than compliance. Whitepapers, PCN's etc.
…On Fri, 28 Oct 2022 at 11:22, Andrew Scheller ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In documentation/index.json
<#2674 (comment)>
:
> @@ -60,14 +60,14 @@
"subpath": "processors.adoc"
},
{
- "title": "PIP",
- "description": "The Product Information Portal (PIP) for Raspberry Pi compliance documents",
+ "title": "Product Information Portal",
+ "description": "Raspberry Pi compliance documents",
Isn't the intention for PIP to (eventually) be more than just compliance
documents?
—
Reply to this email directly, view it on GitHub
<#2674 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA5KWHISEQ5MK25VQTN2LWDWFOSNPANCNFSM6AAAAAARQ5PB6M>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
--
James
|
ghost
reviewed
Oct 28, 2022
aallan
approved these changes
Oct 28, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Spot of rewording, dragged capitalisation into line with the Style Guide, added some consistency.