Skip to content

feat: add isApprovedVcep to Organization + GKS Agent#203

Merged
susannasiebert merged 11 commits intomasterfrom
issue-202-add-vcep-to-org
Jan 5, 2026
Merged

feat: add isApprovedVcep to Organization + GKS Agent#203
susannasiebert merged 11 commits intomasterfrom
issue-202-add-vcep-to-org

Conversation

@katie-perry
Copy link
Contributor

close #202

@katie-perry katie-perry self-assigned this Dec 15, 2025
@katie-perry katie-perry added the enhancement New feature or request label Dec 15, 2025
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry, just remembered this. If we do:

is_approved_vcep: isApprovedVcep

Then we should be able to get the snakecase convention

acoffman
acoffman previously approved these changes Dec 16, 2025
Copy link
Member

@acoffman acoffman left a comment

Choose a reason for hiding this comment

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

LGTM!

@korikuzma
Copy link
Collaborator

I think the cache should be updated to pull in the latest changes (so CI tests pass)

@katie-perry
Copy link
Contributor Author

I think the cache should be updated to pull in the latest changes (so CI tests pass)

I'm having issues updating the cache. I tried generating a new one with civicpy update --hard and it looks like my file ends up being corrupt every time. I'm not really sure how to be able to get a working pkl file to commit here 😭

Does anyone know why I can't generate working pkl files?

Screenshot 2025-12-17 at 4 29 17 PM Screenshot 2025-12-17 at 4 32 28 PM

@katie-perry
Copy link
Contributor Author

I think the cache should be updated to pull in the latest changes (so CI tests pass)

I'm having issues updating the cache. I tried generating a new one with civicpy update --hard and it looks like my file ends up being corrupt every time. I'm not really sure how to be able to get a working pkl file to commit here 😭

Does anyone know why I can't generate working pkl files?

Screenshot 2025-12-17 at 4 29 17 PM Screenshot 2025-12-17 at 4 32 28 PM

FINALLY figured this out - I had a lingering installation that was taking precedence so the correct file was never generating

@susannasiebert susannasiebert merged commit 03910e6 into master Jan 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add isApprovedVcep to Organization

4 participants