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

Sabas merge #105

Merged
merged 126 commits into from
Jan 16, 2023
Merged

Sabas merge #105

merged 126 commits into from
Jan 16, 2023

Conversation

ArcaneDots
Copy link
Contributor

Site seems to be working. Translations should be complete but are rough. Plan to reduce number of translation keys used once this is PR is merged.

sabas and others added 30 commits September 28, 2013 21:10
adding CNAME pointer to su.openstreetmap.it
Suggerito da Ale
Alternative to search by querying nominatim
Screw you json
Atm it's using still Nominatim, the switch occurs changing line 53.
Fixes osmlab#5
I'm the jedi master of typos \o/
Newline missing
Media query to position no text along the logo in narrow screens
@ArcaneDots ArcaneDots marked this pull request as ready for review January 13, 2023 23:17
@ArcaneDots
Copy link
Contributor Author

Fully merged improvements from the Sabas fork back into main project. Fixed a few bugs and verified that notes were generated correctly.

@ArcaneDots
Copy link
Contributor Author

The Sabas fork already had fixes for issues #46 and #47

Copy link
Member

@iandees iandees left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking the time to do this.

Can you please fix a couple things for me:

  • The CNAME, LICENSE, and README.md files should not be marked executable
  • The CNAME file should stay at onosm.org otherwise the site will stop working

CNAME Outdated Show resolved Hide resolved
@ArcaneDots ArcaneDots requested a review from iandees January 16, 2023 05:39
Copy link
Member

@iandees iandees left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@iandees iandees merged commit 529a06b into osmlab:gh-pages Jan 16, 2023
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.

8 participants