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

feat(general): Initial setup for app monitoring tool #8961

Merged
merged 1 commit into from
Jan 13, 2023

Conversation

micieslak
Copy link
Member

@micieslak micieslak commented Jan 4, 2023

What does the PR do

Allows building the app with support for live inspection. When enabled, the separate window is opened to present inspection results. The intention is to track raw data coming from the backend, what can be beneficial for both development and debugging/profiling.

To compile the app with the monitoring support, MONITORING=true should be passed to make.

When compiled with the monitoring support, the monitoring window can be disabled by defining DISABLE_MONITORING_WINDOW env variable.

This is the initial setup, further work is intended to be provided in separate prs.

Closes: #8786
Depends on: status-im/dotherside#85

Kazam_screencast_00098.webm

@status-im-auto
Copy link
Member

status-im-auto commented Jan 4, 2023

Jenkins Builds

Click to see older builds (6)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ f9522a8 #1 2023-01-04 16:22:50 ~5 min imports 📄log
✔️ f9522a8 #1 2023-01-04 16:24:46 ~7 min linux-cpp 📄log
✔️ f9522a8 #1 2023-01-04 16:27:03 ~9 min macos 🍎dmg
✔️ f9522a8 #1 2023-01-04 16:33:35 ~15 min linux 📦tgz
✔️ f9522a8 #1 2023-01-04 16:47:08 ~29 min windows 💿exe
✔️ f9522a8 #1 2023-01-04 16:48:55 ~31 min linux-e2e 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 7ee57e1 #2 2023-01-05 11:30:48 ~4 min imports 📄log
✔️ 7ee57e1 #2 2023-01-05 11:32:17 ~6 min linux-cpp 📄log
✔️ 7ee57e1 #2 2023-01-05 11:35:25 ~9 min macos 🍎dmg
✔️ 7ee57e1 #2 2023-01-05 11:38:52 ~12 min linux 📦tgz
✖️ 7ee57e1 #2 2023-01-05 11:52:12 ~26 min linux-e2e 📄log
✔️ 7ee57e1 #2 2023-01-05 11:54:33 ~28 min windows 💿exe
✔️ 67610e4 #4 2023-01-13 12:19:19 ~3 min linux-cpp 📄log
✔️ 67610e4 #3 2023-01-13 12:20:36 ~4 min tests-nim 📄log
✔️ 67610e4 #4 2023-01-13 12:21:06 ~5 min imports 📄log
✔️ 67610e4 #4 2023-01-13 12:24:19 ~8 min macos 🍎dmg
✔️ 67610e4 #4 2023-01-13 12:25:24 ~9 min linux 📦tgz
✔️ 67610e4 #4 2023-01-13 12:38:41 ~22 min windows 💿exe
✔️ 67610e4 #4 2023-01-13 12:44:49 ~28 min linux-e2e 📄log

Copy link
Contributor

@osmaczko osmaczko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks as good starting point!

Comment on lines +19 to +27
const xhr = new XMLHttpRequest()
xhr.open("GET", "MonitorEntryPoint.qml", false)
xhr.send()

const content = xhr.responseText

if (loader.source != content)
loader.source = content
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😮 🤯

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know, it's poor temporary solution, but it does it's job for now. It will be remastered in the next prs :)

Copy link
Contributor

@alexandraB99 alexandraB99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Configuration for Status app Monitor & Inspection tool
4 participants