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

Update Vortex URL handler #882

Merged
merged 2 commits into from
Aug 23, 2023
Merged

Update Vortex URL handler #882

merged 2 commits into from
Aug 23, 2023

Conversation

termdisc
Copy link
Contributor

Accidentally left a reference to HMM in the Vortex URL handler and an extra $.

Fixing my fix from #880

@termdisc termdisc marked this pull request as ready for review August 23, 2023 15:11
@sonic2kk
Copy link
Owner

Looks good to me, sorry for missing this originally but thanks for the fix! 😄

@sonic2kk sonic2kk merged commit 5d47b13 into sonic2kk:master Aug 23, 2023
@sonic2kk
Copy link
Owner

Also just a side note: This change will probably be left out of the changelog since it's a fix for a fix that wasn't in a release yet. Just in case you're wondering why it's not mentioned. But your work is highly appreciated all the same!

@termdisc
Copy link
Contributor Author

Thanks, and sorry for the mistake.

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.

2 participants