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

Fixed incorrect icon url #436

Merged

Conversation

Ghost-Miner
Copy link
Contributor

Fixed invalid url for the Icon path which i haven't noticed before.

@VictoriousRaptor VictoriousRaptor enabled auto-merge (squash) December 17, 2024 14:56
@VictoriousRaptor
Copy link
Contributor

Hi, please resolve conflicts so we can merge this pr. thx.

auto-merge was automatically disabled December 17, 2024 14:58

Head branch was pushed to by a user without write access

@VictoriousRaptor VictoriousRaptor enabled auto-merge (squash) December 17, 2024 16:04
@VictoriousRaptor VictoriousRaptor merged commit d4f65eb into Flow-Launcher:plugin_api_v2 Dec 17, 2024
5 checks passed
@jjw24 jjw24 added the bug Something isn't working label Dec 18, 2024
@jjw24
Copy link
Member

jjw24 commented Dec 18, 2024

Hmm, thought the validation check would have caught that error.

@Ghost-Miner
Copy link
Contributor Author

Ghost-Miner commented Dec 18, 2024

Hmm, thought the validation check would have caught that error.

I think it would have to check if the URL's HTTP response is 200. to catch this error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants