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

Fix unhandled promise rejection when using markup #68

Merged
merged 1 commit into from
Jul 2, 2023

Conversation

mkalinski
Copy link
Contributor

Method location.output[index].set_use_markup doesn't exist.

But without the exception thrown by its use, the call to location.output[index].set_text overwrites markup settings.

Ensure that only markup or normal text is set, but not both.

Fixes #58

Method `location.output[index].set_use_markup` doesn't exist.

But without the exception thrown by its use, the call to
`location.output[index].set_text` overwrites markup settings.

Ensure that only markup or normal text is set, but not both.
@mkalinski
Copy link
Contributor Author

I took the liberty to remove some other unnecessary calls from the function to make it simpler.

@raujonas
Copy link
Owner

raujonas commented Jul 2, 2023

Thank you very much for creating this PR, it's very appreciated (and thanks for your patience!)

@raujonas raujonas merged commit 4394a3e into raujonas:main Jul 2, 2023
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.

Unhandled promise rejection for commands using <executor.markup.true>
2 participants