Skip to content

Update date.json:added support for edge_mobile_ios#2484

Closed
Scvairy wants to merge 1 commit intomdn:masterfrom
Scvairy:patch-1
Closed

Update date.json:added support for edge_mobile_ios#2484
Scvairy wants to merge 1 commit intomdn:masterfrom
Scvairy:patch-1

Conversation

@Scvairy
Copy link

@Scvairy Scvairy commented Jul 16, 2018

Proof

@ExE-Boss
Copy link
Contributor

ExE-Boss commented Jul 16, 2018

First of all, it would just be edge_ios, second of all, Edge for iOS is part of Safari for iOS just like all browsers on iOS, as it too, uses WebKit via WebView under the hood.

Also, the build will fail.

@ddbeck
Copy link
Contributor

ddbeck commented Jul 18, 2018

Thanks for this PR, @Scvairy! That said, I agree with the comment above that this is covered by the Safari iOS data, so I think it makes sense to close this PR now. Thanks again!

@ddbeck ddbeck closed this Jul 18, 2018
@Elchi3 Elchi3 added the invalid Invalid issues or pull requests (wrong repo, spam, duplicates, etc.). This won't get merged. Sorry! label Jul 19, 2018
@Scvairy Scvairy deleted the patch-1 branch July 19, 2018 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid Invalid issues or pull requests (wrong repo, spam, duplicates, etc.). This won't get merged. Sorry!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants