Skip to content

Add initial permission set for managing your Germ Network account - #12

Merged
ThisIsMissEm merged 2 commits into
mainfrom
feat/permission-sets
Apr 20, 2026
Merged

Add initial permission set for managing your Germ Network account#12
ThisIsMissEm merged 2 commits into
mainfrom
feat/permission-sets

Conversation

@ThisIsMissEm

@ThisIsMissEm ThisIsMissEm commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

The title and detail description might need review for the language that we want to use, but I think this covers the two permissions involved here.

I suspect in the future it may make sense to split out messageMe into it's own collection, such that those can be independently managed by applications without the ability to modify the key package.

Fixes GER-1629

NOTE: In order to use this new permission set, we need to change the CIMD we're using, leaving the legacy atproto transition:generic in place for static.germnetwork.com

Here's a mockup of what the authorization screen would look like with this permission set:
image

@github-actions

github-actions Bot commented Apr 18, 2026

Copy link
Copy Markdown

🟡 Linting Warnings

Warnings:

  • File: lexicons/com/germnetwork/declaration.json
    • com.germnetwork.declaration: bytes field with no maximum size
    • com.germnetwork.declaration: bytes field with no maximum size
    • com.germnetwork.declaration: bytes field with no maximum size

✅ Incompatible Changes

No breaking changes were detected.

✅ DNS

No DNS issues detected!

@anna-germ anna-germ left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cool! Thanks Emelia:)

Comment thread lexicons/com/germnetwork/authManageAccount.json Outdated
@ThisIsMissEm
ThisIsMissEm merged commit 4c81070 into main Apr 20, 2026
2 checks passed
@ThisIsMissEm
ThisIsMissEm deleted the feat/permission-sets branch April 20, 2026 20:18
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