Skip to content

Add City of Vancouver MSDiLOA (dev) OCA Bundle - #197

Merged
esune merged 1 commit into
bcgov:mainfrom
mathieuglaude:cov-msdiloa-oca
Jun 17, 2026
Merged

Add City of Vancouver MSDiLOA (dev) OCA Bundle#197
esune merged 1 commit into
bcgov:mainfrom
mathieuglaude:cov-msdiloa-oca

Conversation

@mathieuglaude

@mathieuglaude mathieuglaude commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

@krobinsonca, @esune

Adds the City of Vancouver — Municipal Services Digital Letter of Authorization (MSDiLOA) OCA Bundle for CANdy Dev, following the same structure as the existing City of Vancouver Rental Property Business Licence bundle.

  • Path: OCABundles/schema/CityOfVancouver/dev-municipal-services-digital-letter-of-authorization/
  • 18 attributes. Card uses authorized_services_summary (primary) + authorized_location_short_address (secondary), reusing the CoV logo/background/colours.
  • Identifiers (CANdy Dev):
    • Schema: YWnESLB4SH275SMNvaJJ1L:2:Municipal Services Digital Letter of Authorization:1.1domain/39764
    • Cred Def: YWnESLB4SH275SMNvaJJ1L:3:CL:39764:Municipal Services Digital Letter of Authorizationdomain/39765
  • Generated with genBundle.sh (OCA Excel Parser 1.2.1) and validated against OCABundles/validators/schema.json.
  • The credential's governance documentation is in Create municipal-services-digital-letter-of-authorization.md DITP#148.

Note on the index files: I added the two new entries to ocabundles.json / ocabundleslist.json by hand (minimal diff). Re-running scripts/gen_ocabundlesjson.sh currently rewrites many unrelated entries (trailing-whitespace + reordering) against the committed index, so I kept it clean — happy to run the full generator instead if you'd prefer.

Signed-off-by: Mathieu Glaude <mathieu@northernblock.io>
@esune

esune commented Jun 17, 2026

Copy link
Copy Markdown
Member

@mathieuglaude thank you. I am not sure why the script reorders the entries, I will have to look into it. That being said, I would let it add the entries even with the reordering so we know the process is consistent.

@swcurran

Copy link
Copy Markdown
Contributor

Almost all looks good, although you can use this as is. The two index files that are included here are auto-generated on merging, so no need to include them. They should probably be removed from the repo as they are not the "source of truth" -- the files generated on merging and stored in the gh-pages branch are that. It was on my to do list that I didn't get to before leaving the team.

Other than that -- all good.

The attribute sorting is done by the Excel-to-Bundle script that we didn't create. I agree that it should not be done, and could be altered. What I would really like to see is removal of the Excel file in favour of YAML, but we're not there yet. That is a tradeoff between (a little) harder for the Bundle Creator (vs. using Excel), but easier for the rest of the GitHub process to see exactly what is happening.

I don't have the power to approve anymore, but :+1 from me.

@krobinsonca
krobinsonca requested a review from esune June 17, 2026 20:10

@esune esune left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @swcurran ! Will approve and merge based on feedback.
I'll look into removing the files from the index and updating teh process/mappings at some point (hopefully soon).

@esune
esune merged commit 6d11a0c into bcgov:main Jun 17, 2026
2 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