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 bank.json New bank add #10556

Merged
merged 4 commits into from
Feb 20, 2025
Merged

Update bank.json New bank add #10556

merged 4 commits into from
Feb 20, 2025

Conversation

Mukhtar27
Copy link
Contributor

@Snowysauce
Copy link
Collaborator

Please stop placing new code at the very end of the file; this is the cause of the failed code checks. For guidance on where the code can/should be placed, please look at my edit to #10544.

also adjusted locationSet for Dubai Islamic Bank per overpass-turbo
@Snowysauce Snowysauce merged commit 26bf05b into osmlab:main Feb 20, 2025
3 checks passed
@Mukhtar27
Copy link
Contributor Author

@Snowysauce , thanks for the help. I am very sorry for adding it like that. Though, i would place the way you have done, do i need to add at the end.
]
}

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