chore: Add span for measuring performance of consolidated api (#37328)
This PR adds spans for fetching consolidated api from client perspective <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Enhanced API call functionality with improved error handling and response validation. - Added tracing for better observability during API interactions. - **Bug Fixes** - Refined error handling for specific scenarios, including unauthorized access. - **Documentation** - Updated function signatures for better clarity on API response handling. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
parent
8cbf8a5504
commit
b24b66f450
|
|
@ -240,11 +240,14 @@ export function* getInitResponses({
|
|||
shouldInitialiseUserDetails,
|
||||
);
|
||||
|
||||
const rootSpan = startRootSpan("fetch-consolidated-api");
|
||||
const initConsolidatedApiResponse: ApiResponse<InitConsolidatedApi> =
|
||||
yield mode === APP_MODE.EDIT
|
||||
? ConsolidatedPageLoadApi.getConsolidatedPageLoadDataEdit(params)
|
||||
: ConsolidatedPageLoadApi.getConsolidatedPageLoadDataView(params);
|
||||
|
||||
endSpan(rootSpan);
|
||||
|
||||
const isValidResponse: boolean = yield validateResponse(
|
||||
initConsolidatedApiResponse,
|
||||
);
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user