Skip to content

Feature/legaltag extension properties

Dadong Zhou requested to merge feature/legaltag_extensionProperties into master

All Submissions:

  • [YES] I have added an explanation of what changes in this merge do and why we should include it?
  • [YES] I have updated the documentation accordingly.
  • [YES] I have added tests to cover my changes.
  • [YES] All new and existing tests passed.
  • [YES] My code follows the code style of this project.
  • [YES] I ran lint checks locally prior to submission.

What is the current behavior?

LegalTag without the 'extensionProperties' field.

Issue: Remember to link the workitem to this pull request.

What is the new behavior?

LegalTag with the optional 'extentionProperties' field.

Does this introduce a breaking change?

  • [NO]

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

Other information

This change is linked to ADR: #30 (closed)

@chad @hmarkovic Please feel free to update the reviewer list.

Thanks.

Merge request reports