refactor(AppHeader): Split responsibilities

grafana / explore-profiles

Explore Profiles is a native Grafana application designed to integrate seamlessly with Pyroscope, the open-source continuous profiling platform, providing a smooth, query-less experience for browsing and analyzing profiling data.

GNU Affero General Public License v3.0

18 stars 1 forks source link

✨ Description

Related issue(s): -

The goal of this PR is to reduce the complexity of the SceneProfilesExplorer class by extracting the logic related to the app header to a separate component. It tries to achieve this by using our architecture guidelines.

📖 Summary of the changes

The main changes are:

Created a new <Header /> component responsible for rendering the header
Extracted the header logic to a new useHeader React hook
Extracted the logic to build a shareable URL and added unit tests

See diff tab for specific comments.

🧪 How to test?

The build should pass

Lines	Statements	Branches	Functions
	11.62% (433/3726)	9.11% (121/1328)	8.6% (102/1186)

Lines

Statements

Branches

Functions

11.62% (433/3726)

9.11% (121/1328)

8.6% (102/1186)

grafana / explore-profiles