feat: add method to add metadata section values as a dictionary (restarted) #470
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.
[Note: this PR replaces #456 (Branch here) which was proving difficult to merge. The PR text is largely unchanged.]
Goal
Adds a
BugsnagMetadata.addMetadataToSection:values:
method to allow merging arbitrary dictionaries with preexisting Metadata.Design
Simple fast enumeration loop over provided values with appropriate checks for:
Conditional addition/deletion depending on whether value is null or not. There is a check to ensure that empty sections are not created for (failing) invalid values. Invalid values are logged and ignored. Delegate callback is called only where there is a material - CRUD - change to the metadata.
Changeset
BugsnagMetadata
.Tests
Review
Outstanding Questions
master
for fixes,next
forfeatures)