Version 5.0 - #156
Merged
Merged
Version 5.0#156
Conversation
Databases are no longer required to have an owner, so the server may return a null ownerId/owner/ownerRef. getDatabases() and getBillingAccountDatabases() now return NA for missing owner fields rather than failing to build the data frame. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
… messages - Add noteFieldSchema() and multipleReferenceFieldSchema(), with schema classes for both types when reading form schemas - Add requiredRule and validationMessage arguments to field schemas and columns to as.data.frame() of form schemas - Exclude notes from getRecords() columns - Support multiple reference fields in importRecords() - Fix hideFromEntry, hideInTable and reviewerOnly, which were not applied correctly to the field schema - Fix user fields being classed as reference fields Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
- compare_recursively() reports a missing field as a single failure, as expect_failure() now requires exactly one failure - Selecting after a filter no longer fails on the server, so check the filtered result instead of expecting an error - Ignore the database tree version in the snapshot comparison, as its value and format depend on the server - Snapshot only the text of the rendered Rmd body, as the rest of the html depends on the pandoc and rmarkdown versions Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
The test servers only return databases with or without an owner depending on their version, so use fixed responses to always test both. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
…udit fields - getDatabases() and getBillingAccountDatabases() failed when the server returned a null description, as the self-managed server does - The lastRecordUpdate column of getBillingAccountDatabases() became logical when no database had records - queryAuditLog() failed on events with none or several resourceTypes; these are now combined into a single comma-separated value Also test these cases with fixed server responses. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
- Limit test ids to 21 characters, the maximum that the self-managed server accepts - Treat null and empty strings as equal in snapshot comparisons, as the self-managed server returns null for empty strings Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Support for the user management changes on the server, new form field types and properties, and test fixes. See
NEWS.mdfor the full release notes.Potential breaking changes:
hideFromEntrynow hides a field from data entry (previously it hid the field from the table),hideInTablenow hides a field from the table (previously ignored), andreviewerOnlynow restricts a field to reviewers (previously ignored)getDatabases()andgetBillingAccountDatabases()returnNAfor the owner columns rather than failinggetRecords()includes them as columns withminimalColumnStyle()New features:
noteFieldSchema()for note fields, which are not included as columns ingetRecords()multipleReferenceFieldSchema()for multiple reference fields, with support inimportRecords()requiredRuleandvalidationMessagearguments for form field schemas, and the corresponding columns inas.data.frame()of a form schemaTesting:
TEST_URLsecret now points at https://pr-1986.activityinfo-testing.org, which runs thefeature-user-managementbranch of the server. It will need to be updated when that environment is removed.🤖 Generated with Claude Code
https://claude.ai/code/session_01MfjP16HSkanSsjZFVy8qPp