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

Added Safari Extention #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

taksh108
Copy link

@taksh108 taksh108 commented Nov 1, 2021

Safari extention added. Works for group Chats but not for Individual chats

Safari extention added. Works for group Chats but not for Individual chats
@tomer8007
Copy link
Owner

tomer8007 commented Nov 1, 2021

Is there any way to prevent the code duplication?
Also do you have an idea of why it doesn't work for individual chats? Do you have a stack trace?

@taksh108
Copy link
Author

taksh108 commented Nov 1, 2021

I actually started learning about making extensions about 24 hours ago, so at this time, I really don't know how to answer your questions. This, and SpotiAds are my first commits to github

@taksh108
Copy link
Author

taksh108 commented Nov 1, 2021

Also, the feature seems to be working fine on both individual and group chats. I must've not tested correctly previousl
Screenshot 2021-11-02 at 1 55 48 AM
y

@tomer8007
Copy link
Owner

This looks good, I think the only problem is the duplication of code files. If you can find a way to not need that and still have the same repo for both Chrome and Safari it'd be great. If not, we can just have a link to your fork for those who want Safari support. The same is true for SpotiAds.

@taksh108
Copy link
Author

taksh108 commented Nov 2, 2021

I have figured out to package It into a single binary so that we only need share only 1 file, but I am not sure if it will work for everyone. It runs on my computer as I built it, but since I am not enrolled in Apple's Paid Developer program, Binaries signed by me might not work for others. Will update once I have researched more. Also, you can check out the directory structure of https://github.com/ajayyy/SponsorBlock as they have all extensions in one repo. Tho the Distribution method is quite different

@vivekwarde
Copy link

Hello @tomer8007 @taksh108 Does restore deleted messages works with Safari WebView 'WKWebView' ?

@tomer8007
Copy link
Owner

@vivekwarde I don't know, never tested it.

@taksh108
Copy link
Author

@vivekwarde this has been too long, I don't remember much tbh. Haven't worked on anything on safari in years

@vivekwarde
Copy link

@tomer8007 @taksh108 In Safari WebView WKStoreKit, WKWebView I have tested that no last seen, no blue tick, no story seen works perfectly but restore deleted messages doesn't works at all. Can you please guide me how to get it solved/working? Its Either some mistake in our implementation or some mistake in your code...

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.

None yet

3 participants