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

Updated for vatSys virtual controller client for VATSIM #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FWright07
Copy link

@FWright07 FWright07 commented Sep 7, 2021

Same as before

@kognise
Copy link
Owner

kognise commented Sep 7, 2021

Does it need a logo?

@kognise kognise changed the title Updated for vatSys Virctual Controller Client for VATSIM Updated for vatSys virtual controller client for VATSIM Sep 7, 2021
@FWright07
Copy link
Author

Yeah, I have the ico file

@FWright07
Copy link
Author

@Lukeos11
Copy link

Lukeos11 commented Oct 10, 2021

Hey,

You need to edit the HTML to add in vatsys, it seems you haven't.

Thanks, Luke

EDIT: Also Discord doesn't accept .ico files you'll need to make it a 512x512 jpg/jpeg or png

@FWright07
Copy link
Author

Ok...
I have edited the HTML, it just hasn't been accepted.
I will add a png File shortly

@kognise
Copy link
Owner

kognise commented Oct 10, 2021

I'll merge as soon as the PNG is included! Thanks :)

EDIT: And please make sure it's included in the README.

@FWright07
Copy link
Author

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.

3 participants