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

[TIMOB-23327] iOS: Introducing onStopBlackListUrl #8019

Closed
wants to merge 2 commits into from

Conversation

AngelkPetkov
Copy link
Contributor

@AngelkPetkov AngelkPetkov commented May 25, 2016

@sgtcoolguy sgtcoolguy changed the title [TIMOB-23327] iOS: Introducing blacklist property [TIMOB-23327] iOS: Introducing blacklist property - WIP DO NOT MERGE May 25, 2016
@AngelkPetkov AngelkPetkov changed the title [TIMOB-23327] iOS: Introducing blacklist property - WIP DO NOT MERGE [TIMOB-23327] iOS: Introducing onStopBlackListUrl and blacklistFirstLoad property May 25, 2016
platforms: [iphone,iphone]
type: Boolean
default: false
since: "5.4.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

As discussed, I don't like that properly. As it also causes parity problems, we should remove it and check every URL that is in the blacklist. If they want to exclude certain domains, they should remove them from the blacklist or set the blacklistedURLs after its creation, which should be possible.

}


if ([self.proxy _hasListeners:@"beforeload"])
Copy link
Collaborator

Choose a reason for hiding this comment

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

Indentation is messed up through the whole new code-block.

@AngelkPetkov AngelkPetkov force-pushed the TIMOB-23327 branch 3 times, most recently from fcfd9de to 78531bf Compare June 2, 2016 18:55
@sgtcoolguy sgtcoolguy added the ios label Jun 8, 2016
@AngelkPetkov AngelkPetkov changed the title [TIMOB-23327] iOS: Introducing onStopBlackListUrl and blacklistFirstLoad property [TIMOB-23327] iOS: Introducing onStopBlackListUrl Jun 10, 2016
@hansemannn hansemannn closed this Dec 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants