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

Issue 3320 - App killed when launching it #17

Closed
wants to merge 10 commits into from

Conversation

dedecej
Copy link
Contributor

@dedecej dedecej commented Dec 14, 2015

Related to issue 3320.

@dedecej
Copy link
Contributor Author

dedecej commented Dec 14, 2015

@fhd: Small fix of incorrect app group id in dev builds.

@@ -26,6 +26,8 @@
static NSString *AdblockPlusDownloadedVersion = @"AdblockPlusDownloadedVersion";
static NSString *AdblockPlusWhitelistedWebsites = @"AdblockPlusWhitelistedWebsites";

static NSString *AdblockPlusSafariExtension = @"AdblockPlusSafariExtension";
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason this is separated from the above by an empty line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wanted to separate this variable, because it has different semantic then values above (it is value and not key used in dictionary).

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry, missed this comment because GitHub didn't send out a notification. Yeah, if it makes sense let's keep it.

@fhd
Copy link
Contributor

fhd commented Dec 14, 2015

@dedecej Thanks! Just one nit.

@fhd
Copy link
Contributor

fhd commented Dec 15, 2015

@dedecej Sorry, missed your comment. LGTM! Merging now.

@fhd
Copy link
Contributor

fhd commented Dec 15, 2015

Merged: 3a7b3aa

@fhd fhd closed this Dec 15, 2015
abpbot pushed a commit that referenced this pull request Jun 21, 2018
Also added/corrected some existing header documentation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants