Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

goat: generic record data #751

Merged
merged 1 commit into from
Sep 8, 2024
Merged

goat: generic record data #751

merged 1 commit into from
Sep 8, 2024

Conversation

bnewbold
Copy link
Collaborator

@bnewbold bnewbold commented Sep 6, 2024

Little hack to make goat work better with generic record data, aka when we don't know the Lexicon. In this context, just dumping the JSON to stdout (pretty printed).

Sharing b/c we might want to do this generally, at least for API endpoints: json.RawMessage instead of lexutil.LexiconTypeDecoder when we encounter unknown schema type. Note that this won't work for records, and the schema rules do allow unknown in records, it just hasn't happened yet.

Another place we encounter this is DID documents, which don't even have $type in the JSON.

@bnewbold bnewbold merged commit f9efd8c into main Sep 8, 2024
7 checks passed
@bnewbold bnewbold deleted the bnewbold/goat-record-data branch September 8, 2024 16:42
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.

2 participants