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

Solving the problem of Missing flag handling in ColorMyWorld #1603 #10

Closed
wants to merge 1 commit into from
Closed

Conversation

yashveeeeer
Copy link

llaske#1603
This is my first pull request to an open source project. I’ve made improvements to handle missing flags in ColorMyWorld, as detailed in issue llaske#1603. Also when i am running the sugarizer using source code i can not see any flags with the country name and also can not see the country flags on the internet version of the sugarizer but the video you have provided shows it, what i have done is while clicking on the "SAINT VINCENT AND THE GRINADINES" it shows the name correctly without showing the undefined part. Please review it and tell me if it is okay, i have also added a video.

Screen.Recording.2024-08-11.224145.mp4

@llaske
Copy link

llaske commented Aug 14, 2024

PR at the wrong place. See llaske#1603 (comment)

@llaske llaske closed this Aug 14, 2024
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