Volodymyr Horobets
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
What has happened with 1.26.2? https://github.com/thoughtspot/visual-embed-sdk/pull/1033
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
You have many alerts in your code. It is a very bad idea to use it. Each one can block the user's interface!
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
Hi guys! Alert for Third-party cookie disabling it is a god idea, but my main problem with alert(DUPLICATE_TOKEN_ERR); and it isn't solved 😦
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
I suggest replacing all alert calls in your SDK with similar code
function customAlert(message) {
if (options.silentErrors !== true) {
alert(message);
} else {
console.error(
The error was hidden, but not resolved, message);
}
}40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
Is it possible solution?
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
init({
thoughtSpotHost: '<%=tshost%>',
authType: AuthType.None,
silentErrors: true
});
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
This happens in the browser, there are no such process.env variables
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
Thanks for understanding!
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
If this is used in a separate system, then this can be done, but in our case it may someday break our entire system and we want to be able to disable it
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
The analytics module is built into our system and it is better for us to have one broken module than the entire broken system.
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
If the analytics module breaks, then our client can survive it, but if the analytics module breaks and then breaks the entire system, then this is already a problem.
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
{silentErrors: true}
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
Give us the option to completely disable alerts when configuring the client
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
Looks like you didn't understand our problem. We generally do not want any secondary module to somehow break our entire system.
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
Maybe it’s not for nothing that eslint complains about this line?
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
I see alert in the new place 😦
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
// eslint-disable-next-line no-alert
alert(DUPLICATE_TOKEN_ERR);
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
The duplicate token problem is a "radio" problem, but it shouldn't break the whole machine!
Third-party libraries should not use alerts because it can completely break the entire system.
40 replies
TFDThoughtSpot For Developers
•Created by Volodymyr Horobets on 11/20/2023 in #dev-help
Hi guys!
Our clients use our product in emergency systems, and if during a critical call, the interface suddenly becomes blocked due to a problem in a secondary analytics module, then this will be a very big problem.
40 replies