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

使用WKWebView來替代已棄用的UIWebView #153

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

Conversation

DDVD233
Copy link
Contributor

@DDVD233 DDVD233 commented Jun 21, 2020

測試過,所有功能應當都是可用的。

@DaidoujiChen
Copy link
Owner

耶? 真假 我以為換 wk 會拿不到 cookie
我試過 kk 就 mer 進去
感謝

@DaidoujiChen
Copy link
Owner

嗯嗯 這邊因為 HTTPCookieStorage.shared.cookies 只拿得到 UIWebView 的部分
WKWebView 的需要從 WKWebsiteDataStore.default().httpCookieStore.getAllCookies
這邊我調整一下

但是可能 target 會變成 11 以上才可以安裝

@DDVD233
Copy link
Contributor Author

DDVD233 commented Jul 3, 2020

這樣嗎。。

我發現了一件很奇怪的事情:我用Simulator測試是拿不到Cookie的,但是如果用Physical Device測試(我用了iPhone X 和 iPad Pro 2018),就可以拿到Cookie。
不管是Simulator還是Physcial Device iOS版本都是13.5,我也不知道為什麼會有差別⋯⋯

抱歉之前沒有進行完善的測試。

@DaidoujiChen
Copy link
Owner

這樣嗎。。

我發現了一件很奇怪的事情:我用Simulator測試是拿不到Cookie的,但是如果用Physical Device測試(我用了iPhone X 和 iPad Pro 2018),就可以拿到Cookie。
不管是Simulator還是Physcial Device iOS版本都是13.5,我也不知道為什麼會有差別⋯⋯

抱歉之前沒有進行完善的測試。

喔喔 沒關係啦
這部分我改下應該就好惹
OwOb 感謝你

@DaidoujiChen
Copy link
Owner

Hello @zjdavid,

溫馨的提醒
因為我不知道你有沒有發現
我有把 pr 丟過去你那邊喔

DDVD233#1

OwOb

@DDVD233
Copy link
Contributor Author

DDVD233 commented Jul 7, 2020

啊⋯⋯抱歉,不知為何Github沒有通知。

我今天回家就看

@DaidoujiChen
Copy link
Owner

3qqq 我這幾天再驗一下
還有不知道是不是大家 iOS 的版本都到 11 以上了

@DDVD233
Copy link
Contributor Author

DDVD233 commented Jul 8, 2020

這也是我擔心的事情。要不這個PR先不要Merge了吧,因為本身優先度也不是很高...

@DaidoujiChen
Copy link
Owner

沒關係 如果要 mer 的話我會再讓大家知道說要更新成新的版本 OwOb
3qqq

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