Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adding new fields in schemas #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mmolimar
Copy link
Contributor

Description of change

I'm adding some missing fields in several entities in their JSON schemas.

Manual QA steps

  • JSON schema validation with the Intuit definition.

Risks

  • Just represents new fields in the JSON schemas.

Rollback steps

  • Revert this branch

@cmerrick
Copy link
Contributor

Hi @mmolimar, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

@cmerrick
Copy link
Contributor

You did it @mmolimar!

Thank you for signing the Singer Contribution License Agreement.

@mmolimar
Copy link
Contributor Author

@@ -420,6 +440,46 @@
},
"DiscountLineDetail": {
"properties": {
"ClassRef": {

Choose a reason for hiding this comment

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

@dbshah1212 Please make a reference to the this object...

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants