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.

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

[IMP] doc: add profiler example #23912

Closed
wants to merge 1 commit into from
Closed

Conversation

Whenrow
Copy link
Contributor

@Whenrow Whenrow commented Mar 28, 2018

The built-in Odoo profiler can be used directly into the logs. This
information was not given in the documentation. This commit add it. An
example on how to use is shown as well as the produced result.

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

The built-in Odoo profiler can be used directly into the logs. This
information was not given in the documentation. This commit add it. An
example on how to use is shown as well as the produced result.
@Yenthe666 Yenthe666 requested a review from mart-e March 28, 2018 06:40
@mart-e mart-e requested review from nseinlet and removed request for mart-e March 28, 2018 06:42
@C3POdoo C3POdoo added the RD research & development, internal work label Mar 28, 2018
@nseinlet
Copy link
Contributor

Hi, you're right, this part is missing. But it seems better to create the page in the 10.0 branch for the 10.0 version, and only show the new profiler in the 11.0 branch.

@Yenthe666
Copy link
Collaborator

Yenthe666 commented Mar 28, 2018

@Whenrow thanks a lot for your efforts!
Can you make two PR's please? One for 10.0 with the 10.0 profiler example and one for 11.0 with the profiler example for V11?
Let us know if something is not unclear!

@mart-e
Copy link
Contributor

mart-e commented Mar 28, 2018

@Yenthe666 you know that @Whenrow is an Odoo's employee, right? 😉

@Yenthe666
Copy link
Collaborator

Ehmm. Now I do. You guys grow too fast 😆

@Whenrow Whenrow closed this Mar 30, 2018
@Whenrow Whenrow deleted the doc-profiler-whe branch March 30, 2018 09:13
@Whenrow
Copy link
Contributor Author

Whenrow commented Mar 30, 2018

This PR is closed. I opened two new ones #23957 for V10 and #23958 for V11.

@sswapnesh
Copy link
Contributor

@Whenrow it's #23958 for v11.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants