Skip to content

Handbook examples#1067

Merged
philbarker merged 7 commits into
masterfrom
HandbookExamples
Jun 23, 2026
Merged

Handbook examples#1067
philbarker merged 7 commits into
masterfrom
HandbookExamples

Conversation

@philbarker

Copy link
Copy Markdown
Collaborator

@jeannekitchens jeannekitchens left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@philbarker it would be useful to include ceterms:statementText to provide context for the dates and related instruction for apprenticeship. Otherwise, looks good.

@philbarker

Copy link
Copy Markdown
Collaborator Author

@jeannekitchens the idea was that the statementType provides the context. That is what every example we have discussed has shown. Could you give an example of what you think the text should say?

Comment thread HandbookExamples/RegulatoryInformation/pell.json
Comment thread HandbookExamples/RegulatoryInformation/pell.json Outdated
@jeannekitchens

Copy link
Copy Markdown
Contributor

@philbarker ok, if we don't already have text for an example, can leave it off but the emphasis was on structured statements and without a statement, it looks odd. Maybe Jared has an example. I'll go ahead and approve.

@philbarker

Copy link
Copy Markdown
Collaborator Author

@jeannekitchens :

but the emphasis was on structured statements and without a statement, it looks odd.

The date is the statement. Remember the property was called statementDate and I suggested changing it to statedDate because it is a "Statement of a date", not "the date of the statement". It's not too late to change the name if you think it looks odd...

@philbarker philbarker merged commit 2506131 into master Jun 23, 2026
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