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

ArmoredChat v3 (DomainChat) (Rebase) #1308

Open
wants to merge 50 commits into
base: master
Choose a base branch
from

Conversation

Armored-Dragon
Copy link
Member

Continued from #1261

  • Settings seem to reset after each launch?

Fixes #1183
Closes #1252
Closes #1253
Fixes #1256
Fixes #1255
Closes #1258
Closes #1254

@Armored-Dragon Armored-Dragon added enhancement New feature or request needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested labels Feb 12, 2025
@Armored-Dragon Armored-Dragon self-assigned this Feb 12, 2025
@Armored-Dragon Armored-Dragon mentioned this pull request Feb 12, 2025
7 tasks
Copy link
Member

@ksuprynowicz ksuprynowicz left a comment

Choose a reason for hiding this comment

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

Thank you! Everything looks fine except indentation in one of the files.

scripts/system/domainChat/formatting.js Outdated Show resolved Hide resolved
@JulianGro

This comment was marked as resolved.

Copy link
Member

@ksuprynowicz ksuprynowicz left a comment

Choose a reason for hiding this comment

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

Everything looks good :)

@Armored-Dragon
Copy link
Member Author

Armored-Dragon commented Feb 18, 2025

Please wait on merging this until the line

console.log("Sending notification to notificationCore:", `Display name: ${displayName}\n Message: ${message}`);
is removed. This was added for @ksuprynowicz to help with bug testing.

@ksuprynowicz
Copy link
Member

In newest version there's no way of selecting the link sent in a message. Selecting it is currently necessary for opening in external browser.

@ksuprynowicz
Copy link
Member

"External window" option in newest version does nothing for me, chat always run internally.
Links are also not wrapped, so they go outside window bounds.

@Armored-Dragon Armored-Dragon added the work in progress Do not merge yet label Feb 18, 2025
@Armored-Dragon
Copy link
Member Author

"External window" option in newest version does nothing for me, chat always run internally.

I can not replicate this on my machine. Currently this PR has debug logs to help figure out why settings are not being loaded on some clients. Please try and open the application and post what the settings are being set to.
It should look something like these two.

[02/18 13:59:46] [DEBUG] [overte.scriptengine] [domainChat.js] Loading settings:  {
[02/18 13:59:46] [DEBUG] [overte.scriptengine]     "enableEmbedding": false,
[02/18 13:59:46] [DEBUG] [overte.scriptengine]     "external_window": false,
[02/18 13:59:46] [DEBUG] [overte.scriptengine]     "join_notification": true,
[02/18 13:59:46] [DEBUG] [overte.scriptengine]     "maximum_messages": 40,
[02/18 13:59:46] [DEBUG] [overte.scriptengine]     "switchToInternalOnHeadsetUsed": true
[02/18 13:59:46] [DEBUG] [overte.scriptengine] }
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml] Got settings:
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml]  {
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml]     "enableEmbedding": false,
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml]     "external_window": false,
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml]     "join_notification": true,
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml]     "maximum_messages": 40,
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml]     "switchToInternalOnHeadsetUsed": true
[02/18 13:59:46] [DEBUG] [qml] [domainChat.qml] }

@ksuprynowicz
Copy link
Member

I also noticed issue with images being bigger that frame of a given message:
overte-snap-by-X74hc595-on-2025-02-18_22-12-03
overte-snap-by-X74hc595-on-2025-02-18_22-12-33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment