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

Feature/Neshan Map #179

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

AmirJabbari
Copy link

Add Neshan Map (nshn.ir)

@AmirJabbari
Copy link
Author

@mattermoran

@mattermoran
Copy link
Owner

hey sorry completely missed it. let me check.
could you also provide a video of it working on both ios and android? for both showing marker and directions. you can use default example app

@@ -72,7 +73,8 @@ private let maps: [Map] = [
Map(mapName: "Naver Map", mapType: MapType.naver, urlPrefix: "nmap://"),
Map(mapName: "Kakao Maps", mapType: MapType.kakao, urlPrefix: "kakaomap://"),
Map(mapName: "TMap", mapType: MapType.tmap, urlPrefix: "tmap://"),
Map(mapName: "Mapy CZ", mapType: MapType.mapyCz, urlPrefix: "szn-mapy://")
Map(mapName: "Mapy CZ", mapType: MapType.mapyCz, urlPrefix: "szn-mapy://"),
Map(mapName: "Neshan Map", mapType: MapType.neshan, urlPrefix: "nshn.ir/")
Copy link
Owner

Choose a reason for hiding this comment

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

"nshn.ir/" does not look like a valid url scheme I don't think it's going to work

Copy link
Author

Choose a reason for hiding this comment

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

I tested it and it works completely, Neshan App doesn't have a custom scheme it has an HTTP deep link

Copy link
Owner

Choose a reason for hiding this comment

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

it may launch it but it won't be shown in the list of available maps.
you sure neshan:// doesn't work?

Copy link
Author

Choose a reason for hiding this comment

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

cbd4bfdd-32ce-4e4b-8d17-7dbf3baa71f2

it shows in the list of available maps

available maps are based on package name I think

Copy link
Owner

Choose a reason for hiding this comment

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

on android yes, I'm talking about iOS

Copy link
Author

Choose a reason for hiding this comment

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

oh sorry I will check this

Copy link
Author

Choose a reason for hiding this comment

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

i fix this

Copy link
Owner

Choose a reason for hiding this comment

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

icon needs to be an svg with an opaque background. you can download a logo from app store and convert to svg

Copy link
Author

Choose a reason for hiding this comment

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

i fix this

@@ -188,6 +189,7 @@ if (await MapLauncher.isMapAvailable(MapType.google)) {
| `.flitsmeister` | ✓ | ✗ | always uses current location | ✗ | ✗ | ✗ | ✗ |
| `.truckmeister` | ✓ | ✗ | always uses current location | ✗ | ✗ | ✗ | ✗ |
| `.sygicTruck` | ✓ | ✗ | always uses current location | ✗ | ✗ | ✗ | ✗ |
| `.neshan` | ✓ | ✗ | ✓ | ✗ | ✓ | ✗ | ✗ |
Copy link
Owner

Choose a reason for hiding this comment

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

could you also add an entry into the Show Marker section

@AmirJabbari
Copy link
Author

@mattermoran Can you please check again?

@AmirJabbari
Copy link
Author

@mattermoran any update?

@AmirJabbari
Copy link
Author

??

@mattermoran
Copy link
Owner

hey. sorry had lots going on.
could you please provide the video of it working on both ios and android? similar to the demo of I have in the readme. I want to make sure it works correctly

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