Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Per diem documentation #374

Merged
merged 18 commits into from
Nov 14, 2023
Merged

Per diem documentation #374

Show file tree
Hide file tree
Changes from 12 commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
9eac72f
added description for per diem expenses spent at
generalpy101 Oct 17, 2023
d1757c2
added per diem fields in expense in schema and fixed a minor type in …
generalpy101 Oct 17, 2023
a86d726
Auto generate API docs
snarayanank2 Oct 17, 2023
3d501ec
merging with main
generalpy101 Oct 18, 2023
4c9ade7
merged main
generalpy101 Oct 18, 2023
e1fde95
removed extra field accidentally added /components/schemas/expense_out
generalpy101 Oct 18, 2023
b3b1975
Auto generate API docs
snarayanank2 Oct 18, 2023
3fcddb7
moved field not required description to spender expense in only
generalpy101 Oct 19, 2023
c98835e
Auto generate API docs
snarayanank2 Oct 19, 2023
b3de81f
minor doc change
generalpy101 Oct 20, 2023
e31ea27
Auto generate API docs
snarayanank2 Oct 20, 2023
1e9508d
added allof to spent_at so that description can be overriden
generalpy101 Oct 24, 2023
0253d6d
minor changes
generalpy101 Oct 25, 2023
42d79df
Merge branch 'main' into per-diem-doc
generalpy101 Oct 25, 2023
5a1e548
minor changes
generalpy101 Oct 25, 2023
d9540a9
Auto generate API docs
snarayanank2 Oct 25, 2023
25c3b08
merging with main
generalpy101 Nov 7, 2023
db652de
merging with main
generalpy101 Nov 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion reference/admin.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3442,7 +3442,7 @@ components:
type: integer
nullable: true
description: |
Specific to per diem related expenses, number of days ofr which per diem claim is made.
Specific to per diem related expenses, number of days for which per diem claim is made.
example: 2
started_at:
$ref: '#/components/schemas/timestamptz_utc'
Expand Down
2 changes: 1 addition & 1 deletion reference/approver.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1516,7 +1516,7 @@ components:
type: integer
nullable: true
description: |
Specific to per diem related expenses, number of days ofr which per diem claim is made.
Specific to per diem related expenses, number of days for which per diem claim is made.
example: 2
started_at:
$ref: '#/components/schemas/timestamptz_utc'
Expand Down
21 changes: 20 additions & 1 deletion reference/spender.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1660,7 +1660,7 @@ components:
type: integer
nullable: true
description: |
Specific to per diem related expenses, number of days ofr which per diem claim is made.
Specific to per diem related expenses, number of days for which per diem claim is made.
example: 2
started_at:
$ref: '#/components/schemas/timestamptz_utc'
Expand Down Expand Up @@ -2016,6 +2016,10 @@ components:
$ref: '#/components/schemas/id_string'
spent_at:
$ref: '#/components/schemas/spent_at'
description: |
Date on which the expense was made.
This date cannot be in future.
This field is not required for creating Per Diem Expenses
source:
$ref: '#/components/schemas/source'
merchant:
Expand Down Expand Up @@ -2074,6 +2078,21 @@ components:
$ref: '#/components/schemas/locations'
custom_fields:
$ref: '#/components/schemas/custom_fields'
per_diem_rate_id:
allOf:
- $ref: '#/components/schemas/fk_integer'
nullable: true
example: 583
description: |
Specific to per diem expense. This represents the per diem's rate id given by Fyle
during creation of per diem rate for which this expense is created.
per_diem_num_days:
allOf:
- $ref: '#/components/schemas/fk_integer'
nullable: true
example: 2
description: |
Specific to per diem related expenses, number of days for which per diem claim is made.
expense_add_comment_action_in:
type: object
required:
Expand Down
26 changes: 23 additions & 3 deletions src/components/schemas/expense.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ expense_out:
type: integer
nullable: true
description: |
Specific to per diem related expenses, number of days ofr which per diem claim is made.
Specific to per diem related expenses, number of days for which per diem claim is made.
example: 2
started_at:
$ref: './fields.yaml#/timestamptz_utc'
Expand Down Expand Up @@ -649,7 +649,7 @@ expense_in:
type: integer
nullable: true
description: |
Specific to per diem related expenses, number of days ofr which per diem claim is made.
Specific to per diem related expenses, number of days for which per diem claim is made.
example: 2
started_at:
$ref: './fields.yaml#/timestamptz_pst'
Expand Down Expand Up @@ -699,7 +699,12 @@ spender_expense_in:
id:
$ref: './fields.yaml#/id_string'
spent_at:
$ref: './fields.yaml#/spent_at'
allOf:
- $ref: './fields.yaml#/spent_at'
description: |
Date on which the expense was made.
This date cannot be in future.
This field is not required for creating Per Diem Expenses
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we talk about how per diems are created/most probably in the header?

source:
$ref: './fields.yaml#/source'
merchant:
Expand Down Expand Up @@ -758,6 +763,21 @@ spender_expense_in:
$ref: './fields.yaml#/locations'
custom_fields:
$ref: './fields.yaml#/custom_fields'
per_diem_rate_id:
allOf:
- $ref: './fields.yaml#/fk_integer'
nullable: true
example: 583
description: |
Specific to per diem expense. This represents the per diem's rate id given by Fyle
during creation of per diem rate for which this expense is created.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add note that this is required in case of per diem category.

per_diem_num_days:
allOf:
- $ref: './fields.yaml#/fk_integer'
nullable: true
example: 2
description: |
Specific to per diem related expenses, number of days for which per diem claim is made.
Copy link
Contributor

Choose a reason for hiding this comment

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

Add note that this is required for per diems


admin_expense_in:
allOf:
Expand Down
Loading