Skip to content

APS-4241 Initial Draft of Getting Started content#1

Merged
phowells merged 4 commits intomainfrom
APS-4241_getting-started
Mar 27, 2026
Merged

APS-4241 Initial Draft of Getting Started content#1
phowells merged 4 commits intomainfrom
APS-4241_getting-started

Conversation

@phowells
Copy link
Collaborator

@phowells phowells requested a review from dstainton February 25, 2026 20:13
@phowells phowells force-pushed the APS-4241_getting-started branch from 22c5748 to 31e123e Compare February 25, 2026 23:49
@phowells phowells requested a review from rustyjux February 25, 2026 23:49
@phowells phowells force-pushed the APS-4241_getting-started branch from 5db9598 to 5c0e65e Compare February 26, 2026 17:15

5. Certified datasets are monitored and maintained over time.

Authoritative certification is not a one-time approval. It requires ongoing oversight.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

designation

Copy link

@dstainton dstainton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a few comments related to authoritative data - 'certification' is now 'designation' so all references to certification/certified etc. will need to be updated. Otherwise it's good to go.

Copy link

@rustyjux rustyjux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall. I only skimmed the content and have not given feedback on it, since I assume that wasn't why I was tagged here.

I left a few comments re: conventions - most are opinionated and not required changes.

@@ -0,0 +1,74 @@
# Getting started with Connected Services
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like an odd convention to have a constant level one title (though maybe was in Tanya's specification). This is contrary to what we (APS) and DevEx do in their docs, but I guess it's fine if that's the convention we want to follow for this project.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at the word doc draft, I don't see this as a requirement. I'd suggest removing this line from all pages and promoting all headings up one level.

And I would reinforce my other recommendation to specify a title in the front matter:

---
title: Authoritative Data Register (ADR)
---

this title will show in the sidebar nav and top of the page as H1. If you want a different title to display as H1 (e.g. a longer title than in the sidebar nav), then you can include a level one heading.

It's sort of up to you if you want to keep the H1 in each doc duplicating what is the in the title, but for APS docs we omitted it and just use the front matter.

- About us: about-us.md
- Getting started:
- Connected Services overview: getting-started.md
- Purpose and value: getting-started/purpose-and-value.md
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a huge deal but to me this is an (annoying) anti-pattern. Instead of needing to specify the title twice, just use the level one header from the document or specify a title in the front matter, e.g. as in https://github.com/bcgov/aps-infra-platform/blob/test/documentation/how-to/create-gateway.md?plain=1#L1C1-L3C4

Suggested change
- Purpose and value: getting-started/purpose-and-value.md
- getting-started/purpose-and-value.md

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a heads up that regardless of what is here in mkdocs.yml, or in the H1 or front-matter title, the HTML page <title> comes from the path / file name.

So really my suggestion is to keep the page titles within the docs, as opposed to duplicating here in mkdocs.yml.

**Primary audience**

- Developers building integrations

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blank lines between list items seems like a bad convention. It is harder to read the markdown and (more importantly) adds extra vertical spacing in the rendered output, making the list look 'loose'.

Do keep a blank line before the list.

@@ -1,14 +1,4 @@
# Connected Services TechDocs
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid 'TechDocs' - the meaning is intuitive enough, but no one needs to know that TechDocs is powering the backend here.

I'd suggest 'Documentation' or 'Technical Documentation'.

For comparison, see other docs in https://developer.gov.bc.ca/docs

@phowells phowells dismissed dstainton’s stale review March 27, 2026 15:39

This is not the source of truth for the content yet so making changes here will only be overwritten with the next update.

@phowells phowells merged commit bf67357 into main Mar 27, 2026
3 checks passed
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.

3 participants