[ADD] Expenses: Expense cards #15154
Conversation
|
|
larm-odoo
force-pushed
the
19.0-expenses-expense-card-larm
branch
2 times, most recently
from
1dcee2c to
7767125
Compare
larm-odoo
force-pushed
the
19.0-expenses-expense-card-larm
branch
from
7767125 to
dc7c7d2
Compare
|
Hello @Laura-labe - I have completed my first draft of the expense card doc. I cannot test this yet, so I only have one screenshot. I wanted to make sure what I have in here is accurate, as my database can only show me so much, without actually being connected and making transactions. |
Laura-labe
left a comment
Laura-labe
left a comment
There was a problem hiding this comment.
Hello @larm-odoo , thank you for your nice work :) I just reviewed the documentation and added some changes requests. Could you please have a look? Thank you!
PS: Feel free to rephrase any of the suggestions I provided, they’re intended to convey the key ideas I’d like to highlight.
larm-odoo
force-pushed
the
19.0-expenses-expense-card-larm
branch
from
dc7c7d2 to
2f71801
Compare
|
Thank you SO much @Laura-labe! Would you please check all my changes and make sure there is nothing else you want added? Thank yoU! |
|
Hello @larm-odoo , thank you for your quick changes :) I added 2 small comments about the Stripe set up and the IOS availability of the virtual card. After those modifications, we'll be good to go. Thank you! |
larm-odoo
force-pushed
the
19.0-expenses-expense-card-larm
branch
from
2f71801 to
ef130d4
Compare
|
Thank you so much @Laura-labe! I've implemented the changes, and am sending this off for final review so it can be published. |
|
Hi @Felicious - this has been revoewed and approved by Laura (LABE) so this is ready for a final tech review. Thank you! |
Felicious
left a comment
Felicious
left a comment
There was a problem hiding this comment.
Wonderful work on this PR, @larm-odoo !
Approving with a really minor and optional nitpick (:
@robodoo delegate+
larm-odoo
force-pushed
the
19.0-expenses-expense-card-larm
branch
from
ef130d4 to
46e216f
Compare
|
@robodoo r+ |
4 participants
New expense cards created - documentation needed. This was requested on this task card.
Original task card for this doc.