Skip to content

[18.0][IMP] usability_api_connector: configurable log limit, latest api log#306

Merged
Saran440 merged 1 commit into
18.0from
18.0-usability_api_connector-log_limit-latest_api_log
Jul 19, 2026
Merged

[18.0][IMP] usability_api_connector: configurable log limit, latest api log#306
Saran440 merged 1 commit into
18.0from
18.0-usability_api_connector-log_limit-latest_api_log

Conversation

@ROBBYHOOD9

Copy link
Copy Markdown
Contributor
  • remove limit def _truncate_text and add api_connector.log_limit to limit config payload
  • add helper _get_latest_api_log() in common.base.api to get latest api call

… helper

- _truncate_text: limit now read from api_connector.log_limit system
  parameter (0 = keep full payload/result, new default) instead of the
  hardcoded 1000 chars that silently lost log data
- add _get_latest_api_log: deterministic latest-log lookup (create_date,
  id) for downstream modules
@Saran440

Copy link
Copy Markdown
Collaborator

Hi @Saran440,
some modules you are maintaining are being modified, check this out!

@ROBBYHOOD9 ROBBYHOOD9 changed the title [IMP] usability_api_connector: configurable log limit, latest api log [18.0][IMP] usability_api_connector: configurable log limit, latest api log Jul 19, 2026

@Saran440 Saran440 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Code Review 👍

@Saran440

Copy link
Copy Markdown
Collaborator

/ocabot merge patch

@Saran440

Copy link
Copy Markdown
Collaborator

What a great day to merge this nice PR. Let's do it!
Prepared branch 18.0-ocabot-merge-pr-306-by-Saran440-bump-patch, awaiting test results.

@Saran440
Saran440 merged commit 5d38b66 into 18.0 Jul 19, 2026
4 checks passed
@Saran440

Copy link
Copy Markdown
Collaborator

Congratulations, your PR was merged at 797bc0e. Thanks a lot for contributing to ecosoft-odoo. ❤️

@Saran440
Saran440 deleted the 18.0-usability_api_connector-log_limit-latest_api_log branch July 19, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants