Skip to content

IE - OX 7968- Expense Budget Attributes Update to include all publicly documented attributes#260

Merged
ananta0799 merged 1 commit into
masterfrom
OX-7968-mavenlink-gem-update-expense-budget-specification
Dec 5, 2025
Merged

IE - OX 7968- Expense Budget Attributes Update to include all publicly documented attributes#260
ananta0799 merged 1 commit into
masterfrom
OX-7968-mavenlink-gem-update-expense-budget-specification

Conversation

@ananta0799
Copy link
Copy Markdown
Contributor

@ananta0799 ananta0799 commented Dec 4, 2025

Updated the expense_budgets specification to include all publicly documented attributes, create_attributes, and update_attributes.

@ananta0799 ananta0799 self-assigned this Dec 4, 2025
Copy link
Copy Markdown

@juni-t juni-t left a comment

Choose a reason for hiding this comment

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

LGTM, but maybe someone with a little more context on this should also approve.

Copy link
Copy Markdown
Contributor

@b6b b6b left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ajitkantata
Copy link
Copy Markdown
Contributor

I don't think it will create a problem as its used in sum but while getting the Expense budget we get the fields differently https://developer.kantata.com/tag/Expense-Budgets#operation/get-expense-budgets!in=query&path=optional_fields&t=request. Will that create a problem not sure?

@b6b
Copy link
Copy Markdown
Contributor

b6b commented Dec 4, 2025

@ajitkantata It will not cause issues but there is the requirement of ensuring the optional fields are specified in the request to get the data back in the response. So these fields will now be accessible if the optional fields are specified in the request, otherwise they will not have data.

Copy link
Copy Markdown
Contributor

@ajitkantata ajitkantata left a comment

Choose a reason for hiding this comment

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

LGTM

@ananta0799 ananta0799 merged commit d697719 into master Dec 5, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants