Skip to content

Conversation

@larm-odoo
Copy link
Contributor

Date format changed,  updated minor formatting changes, and updated all pics.

Original task card for this PR.

@larm-odoo larm-odoo self-assigned this Jan 21, 2026
@larm-odoo larm-odoo added the 2 label Jan 21, 2026
@robodoo
Copy link
Collaborator

robodoo commented Jan 21, 2026

Pull request status dashboard

@C3POdoo C3POdoo requested a review from a team January 21, 2026 22:35
@larm-odoo larm-odoo requested a review from huisit January 21, 2026 22:35
@larm-odoo
Copy link
Contributor Author

Hi @huisit - this is ready for a peer review. Thanks!

Copy link
Contributor

@huisit huisit left a comment

Choose a reason for hiding this comment

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

@larm-odoo I shared some optional suggestions, take it if it resonates :)

I think it might be helpful to include a few collapsed transaction groups in the control-accounts.png screenshot at the end of the article to illustrate what the "default view" looks like, but is there a reason you expanded all of the transactions?

@larm-odoo larm-odoo force-pushed the 19.0-lunch-update-manage-accounts-larm branch from b4a2592 to 0d9f24f Compare January 22, 2026 22:38
@larm-odoo larm-odoo requested a review from Felicious January 22, 2026 22:39
@larm-odoo
Copy link
Contributor Author

Hi @Felicious - this is ready for a final/tehc review. Thanks!

Copy link
Contributor

@Felicious Felicious left a comment

Choose a reason for hiding this comment

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

Looks good! @larm-odoo

@robodoo delegate+

@larm-odoo larm-odoo force-pushed the 19.0-lunch-update-manage-accounts-larm branch from 0d9f24f to 23f548c Compare January 27, 2026 14:12
@larm-odoo
Copy link
Contributor Author

@robodoo r+

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants