[Analytics] Rename `SWITCH_REPORT` to `OPEN_REPORT`

[kacper-mikolajczak]

Intro

We've been fiddling around with making SWITCH_REPORT more fine-grained in order to get clearer perspective on the timing.

When investigating the timing use cases, most of the times we use it to measure general time taken to open report, not just switching the report.

For better connection between actual use case and definition, we could rename SWITCH_REPORT to OPEN_REPORT.

The changes would look like so:
SWITCH_REPORT -> OPEN_REPORT
SWITCH_REPORT_THREAD -> OPEN_REPORT_THREAD
SWITCH_REPORT_FROM_PREVIEW -> OPEN_REPORT_FROM_PREVIEW

[kacper-mikolajczak]

CC @mountiny @adhorodyski

[mountiny]

Alright I think this is ok but we might have to use both for migration step if we want to change this for the performance too / not only timings

[kacper-mikolajczak]

Right, wondering if we could detach perf timings, constants and analytics from each other, as they are now all under TIMING umbrella:

App/src/CONST.ts

Lines 965 to 989 in a70e9be

	TIMING: {
	CALCULATE_MOST_RECENT_LAST_MODIFIED_ACTION: 'calc_most_recent_last_modified_action',
	CHAT_FINDER_RENDER: 'search_render',
	CHAT_RENDER: 'chat_render',
	OPEN_REPORT: 'open_report',
	HOMEPAGE_INITIAL_RENDER: 'homepage_initial_render',
	REPORT_INITIAL_RENDER: 'report_initial_render',
	SWITCH_REPORT: 'switch_report',
	SWITCH_REPORT_FROM_PREVIEW: 'switch_report_from_preview',
	SWITCH_REPORT_THREAD: 'switch_report_thread',
	SIDEBAR_LOADED: 'sidebar_loaded',
	LOAD_SEARCH_OPTIONS: 'load_search_options',
	COLD: 'cold',
	WARM: 'warm',
	REPORT_ACTION_ITEM_LAYOUT_DEBOUNCE_TIME: 1500,
	SHOW_LOADING_SPINNER_DEBOUNCE_TIME: 250,
	TEST_TOOLS_MODAL_THROTTLE_TIME: 800,
	TOOLTIP_SENSE: 1000,
	TRIE_INITIALIZATION: 'trie_initialization',
	COMMENT_LENGTH_DEBOUNCE_TIME: 500,
	SEARCH_OPTION_LIST_DEBOUNCE_TIME: 300,
	RESIZE_DEBOUNCE_TIME: 100,
	UNREAD_UPDATE_DEBOUNCE_TIME: 300,
	SEARCH_FILTER_OPTIONS: 'search_filter_options',
	},

It might be cumbersome at the beginning but should be worth to do, considering we would like to invest more into analytics in the future.

Also, as the monitoring grows in complexity, do we have a documentation of what each metric represents?

P.S.

There's already a OPEN_REPORT const which I missed in original message. In that case, do you know what is stopping us from migrating SWITCH_REPORT to OPEN_REPORT entirely? (e.g. analytics setup or some external systems)

[mountiny]

I think we might have to make some changes in graphana to rename that

Otherwise all for making it clearer

And I dont think there is much documentation about the metrics yet so it might be worth creating some markdown as you work on it

[adhorodyski]

+1 on the docs for this! It'd be only more and more valuable for us to get a common understanding of what exactly each metric represents.

[melvin-bot]

This issue has not been updated in over 15 days. @mountiny, @kacper-mikolajczak eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!