Skip to content

Conversation

@ateucher
Copy link
Contributor

@ateucher ateucher commented Aug 26, 2025

This gives an informative error if you call call_agenda() with a cohort_id that doesn't match a cohort name in the registry.

Now I get:

registry_url <- "the-registry-url"

call_agenda(registry_url = registry_url, 
                cohort_id = "not-a-cohort", 
                call_number = 2)

#> ✔ Reading from "OpenscapesChampionsCohortRegistry".
#> ✔ Range ''cohort_metadata''.
#> New names:
#> Error in `call_agenda()`:
#> ! `cohort_id` "not-a-cohort" not found in cohort registry. Check
#>   `cohort_id` against cohort names in registry.

Created on 2025-08-26 with reprex v2.1.1

Closes #117

Copy link
Member

@stefaniebutland stefaniebutland left a comment

Choose a reason for hiding this comment

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

Tested. Works - informatively - for me thank you!

Copy link
Contributor

@seankross seankross left a comment

Choose a reason for hiding this comment

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

lgtm

@ateucher ateucher merged commit dcc5a55 into main Sep 3, 2025
6 checks passed
@ateucher ateucher deleted the check-cohort-name branch September 3, 2025 17:31
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.

Better error message for Error knitting README.Rmd

4 participants