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.
Seeing reporting of 'Unable to resolve link' errors for items that are included in the workbook that should be ignored as errors on dry run as they will be posted before the item that is linking to it when actually posting.
I inadvertently introduced this for cases where the alias was provided as the 'error_id' parameter to the error_report function when attempting to fix the bug that prevented reporting of other types of validation errors in an informative way.
The issue was due to the error responses including the item type concatenated with the alias string, and now because of removal of the clause to use the alias if provided as a parameter in order to make the error reporting more informative there needs to be additional parsing to get the alias bit and just that with no extra text.
{'location': 'body', 'name': 'Schema: ', 'description': 'Unable to resolve link: /Biosample/test-old-dekker-lab:Biosample_hap1cellsT2_HAP1-XH-G1-SD20221126-ED20221203'}
Is an example of the returned response and the change here will remove the /Biosample/ part so only the alias remains to compare with other aliases in the workbook.