Skip to content

Conversation

@larm-odoo
Copy link
Contributor

Updated configuration menu - more items needed to be added to the config section, and I removed some sections ot be added to their own docs (since the doc was getting very long)

Original task card for this PR.

@robodoo
Copy link
Collaborator

robodoo commented Jan 22, 2026

Pull request status dashboard

@larm-odoo larm-odoo self-assigned this Jan 22, 2026
@larm-odoo larm-odoo added the 2 label Jan 22, 2026
@larm-odoo larm-odoo requested a review from huisit January 22, 2026 21:24
@larm-odoo
Copy link
Contributor Author

Hi @huisit - this is ready for a peer review! I added missing info, and took out sections that I am putting into their own separate docs.

@C3POdoo C3POdoo requested a review from a team January 22, 2026 21:25
@larm-odoo larm-odoo force-pushed the 19.0-attendances-update-main-doc-larm branch from 7fa978e to 440cc5e Compare January 22, 2026 22:08
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 LGTM!

@larm-odoo larm-odoo force-pushed the 19.0-attendances-update-main-doc-larm branch from 440cc5e to 56670e7 Compare January 23, 2026 16:23
@larm-odoo larm-odoo requested a review from Felicious January 23, 2026 16:23
@larm-odoo
Copy link
Contributor Author

Thank you for the review @huisit! I completely forgot about the italicizing, and you caught some stuff that needed to change because of edits and changes ot the app that I missed- I appreciate the fine-tooth comb review!

@larm-odoo
Copy link
Contributor Author

@Felicious - this is ready for a final/tech 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.

@robodoo delegate=larm-odoo

@larm-odoo larm-odoo force-pushed the 19.0-attendances-update-main-doc-larm branch from 56670e7 to c238719 Compare January 25, 2026 00:46
@larm-odoo
Copy link
Contributor Author

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Jan 25, 2026

@larm-odoo because this PR has multiple commits, I need to know how to merge it:

  • merge to merge directly, using the PR as merge commit message
  • rebase-merge to rebase and merge, using the PR as merge commit message
  • rebase-ff to rebase and fast-forward

@larm-odoo
Copy link
Contributor Author

@robodoo merge

@robodoo
Copy link
Collaborator

robodoo commented Jan 25, 2026

Merge method set to merge directly, using the PR as merge commit message.

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