Skip to content

Studio javascript error for a warning #660

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.

Sign up for GitHub

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

Jump to bottom
Closed
isc-etamarch opened this issue Dec 4, 2024 · 0 comments · Fixed by #684
Closed

Studio javascript error for a warning #660

isc-etamarch opened this issue Dec 4, 2024 · 0 comments · Fixed by #684
Assignees
@isc-etamarch
Labels
bug Something isn't working customer Based on customer feedback (as opposed to something identified by developers) usability
Milestone
H1 2025 Targets

Comments

@isc-etamarch
Copy link
Collaborator

In line 169 of git-webui.js, warningCallback is referenced, which hasn't been explicitly defined anywhere earlier. In studio this causes an error, meaning the warning never gets printed out to the UI.
@isc-etamarch isc-etamarch added bug Something isn't working customer Based on customer feedback (as opposed to something identified by developers) usability labels Dec 4, 2024
@isc-tleavitt isc-tleavitt added this to the H1 2025 Targets milestone Dec 13, 2024
@isc-etamarch isc-etamarch self-assigned this Jan 10, 2025
@isc-etamarch isc-etamarch mentioned this issue Jan 13, 2025
Merged
@isc-etamarch isc-etamarch linked a pull request Jan 13, 2025 that will close this issue
Fix Studio warningCallback issue #684
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@isc-etamarch isc-etamarch

Labels
bug Something isn't working customer Based on customer feedback (as opposed to something identified by developers) usability
Projects
None yet
Milestone
H1 2025 Targets
Development

Successfully merging a pull request may close this issue.

Fix Studio warningCallback issue intersystems/git-source-control
2 participants
@isc-tleavitt @isc-etamarch