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

fix: Move Monitor dump to after_response #26703

Merged
merged 2 commits into from
Jun 6, 2024
Merged

Conversation

ankush
Copy link
Member

@ankush ankush commented Jun 6, 2024

It needs access to response to capture status code and size etc

It needs access to response to capture status code and size etc
@ankush ankush requested review from a team and akhilnarang and removed request for a team June 6, 2024 12:57
@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label Jun 6, 2024
@ankush ankush enabled auto-merge (squash) June 6, 2024 12:58
@ankush ankush added the backport version-15-hotfix Backport the PR to v15 label Jun 6, 2024
This was just not tracked before as it happened post response
@ankush ankush merged commit 04642ed into frappe:develop Jun 6, 2024
23 checks passed
@ankush ankush deleted the monitor branch June 6, 2024 14:38
mergify bot pushed a commit that referenced this pull request Jun 6, 2024
It needs access to response to capture status code and size etc

(cherry picked from commit 04642ed)
ankush added a commit that referenced this pull request Jun 7, 2024
It needs access to response to capture status code and size etc

(cherry picked from commit 04642ed)

Co-authored-by: Ankush Menat <ankush@frappe.io>
frappe-pr-bot pushed a commit that referenced this pull request Jun 11, 2024
## [15.29.2](v15.29.1...v15.29.2) (2024-06-11)

### Bug Fixes

* Append symbol on number card ([#26731](#26731)) ([#26732](#26732)) ([38457a5](38457a5))
* apply perms on letter head selection (backport [#26563](#26563)) ([#26583](#26583)) ([ef4bb3b](ef4bb3b))
* Email tracking without "use_ssl" ([#26718](#26718)) ([#26735](#26735)) ([f78d8f8](f78d8f8))
* Move Monitor dump to after_response ([#26703](#26703)) ([#26709](#26709)) ([c24ee13](c24ee13))
* **package_release:** set path to tarball ([9606348](9606348))
* recursive fetch from causes infinite loop ([#26695](#26695)) ([#26697](#26697)) ([9c33352](9c33352))
* Skip letter heads if no permission ([#26689](#26689)) ([#26692](#26692)) ([c9b5df3](c9b5df3))
* Skip virtual fields in all select queries ([#26700](#26700)) ([#26702](#26702)) ([3797f8f](3797f8f))
* **UX:** Disable number card filters on standard ([#26676](#26676)) ([#26680](#26680)) ([58f154f](58f154f))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
add-test-cases Add test case to validate fix or enhancement backport version-15-hotfix Backport the PR to v15
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant