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

Breakage report improvements #1947

Closed
wants to merge 27 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
4f6b661
BSK pointing to feature branch, Privacy dashboard VC updated
federicocappelli Nov 16, 2023
626bc55
Merge branch 'develop' into fcappelli/breakage_report_improvements
federicocappelli Nov 20, 2023
358c224
BSK updated
federicocappelli Nov 20, 2023
2913aad
Merge branch 'develop' into fcappelli/breakage_report_improvements
federicocappelli Nov 23, 2023
1898b72
BSK updated to 85.0.0
federicocappelli Nov 23, 2023
fd8f9c3
BSK from branch
federicocappelli Nov 27, 2023
2639382
Merge branch 'develop' into fcappelli/breakage_report_improvements
federicocappelli Nov 27, 2023
d3c0064
Merge branch 'develop' into fcappelli/breakage_report_improvements
federicocappelli Nov 28, 2023
aaf86bd
BSK updated
federicocappelli Nov 28, 2023
16a77a6
cleanup #1
federicocappelli Dec 1, 2023
cb95a77
Merge branch 'main' into fcappelli/breakage_report_improvements_2
federicocappelli Dec 7, 2023
5046735
web report broken website is now used everywhere in the app
federicocappelli Dec 7, 2023
7fc972a
string updated
federicocappelli Dec 7, 2023
37b0611
Merge branch 'main' into fcappelli/breakage_report_improvements_2
federicocappelli Dec 7, 2023
03e363e
cleanup and new menu icon
federicocappelli Dec 11, 2023
a618f2f
Merge branch 'main' into fcappelli/breakage_report_improvements_2
federicocappelli Dec 11, 2023
46c2da1
string and swiftlint cleanup
federicocappelli Dec 11, 2023
865d7d6
Merge branch 'main' into fcappelli/breakage_report_improvements_2
federicocappelli Dec 11, 2023
75fb570
swiftlint re-aneabled
federicocappelli Dec 11, 2023
ae1f0ee
warning fixed
federicocappelli Dec 11, 2023
1b3fa93
lint fixes
federicocappelli Dec 11, 2023
6e49071
fixed unit tests
federicocappelli Dec 12, 2023
dec7ab7
Merge branch 'main' into fcappelli/breakage_report_improvements_2
federicocappelli Dec 12, 2023
92a2670
Merge branch 'main' into fcappelli/breakage_report_improvements_2
federicocappelli Dec 13, 2023
0d5d83e
done wired to report broken site
federicocappelli Dec 14, 2023
0d7845d
Merge branch 'main' into fcappelli/breakage_report_improvements_2
federicocappelli Dec 20, 2023
bb8b8e8
updated versions
Jan 9, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 13 additions & 5 deletions DuckDuckGo.xcodeproj/project.pbxproj
Original file line number Diff line number Diff line change
Expand Up @@ -6345,8 +6345,6 @@
children = (
AA3D531A27A2F57E00074EC1 /* Feedback.swift */,
AA3D531C27A2F58F00074EC1 /* FeedbackSender.swift */,
AAD8078627B3F45600CF7703 /* WebsiteBreakage.swift */,
AAD8078427B3F3BE00CF7703 /* WebsiteBreakageSender.swift */,
);
path = Model;
sourceTree = "<group>";
Expand Down Expand Up @@ -7863,9 +7861,9 @@
B6FA893A269C414900588ECD /* PrivacyDashboard */ = {
isa = PBXGroup;
children = (
F1D6DA032B21E90500EE9745 /* Breakage */,
B6FA893B269C41ED00588ECD /* View */,
1E7E2E8F29029A2A00C01B54 /* ContentBlockingRulesUpdateObserver.swift */,
1E7E2E9129029F9B00C01B54 /* WebsiteBreakageReporter.swift */,
1E7E2E932902AC0E00C01B54 /* PrivacyDashboardPermissionHandler.swift */,
);
path = PrivacyDashboard;
Expand Down Expand Up @@ -7942,6 +7940,16 @@
path = JSAlert;
sourceTree = "<group>";
};
F1D6DA032B21E90500EE9745 /* Breakage */ = {
isa = PBXGroup;
children = (
1E7E2E9129029F9B00C01B54 /* WebsiteBreakageReporter.swift */,
AAD8078627B3F45600CF7703 /* WebsiteBreakage.swift */,
AAD8078427B3F3BE00CF7703 /* WebsiteBreakageSender.swift */,
);
path = Breakage;
sourceTree = "<group>";
};
/* End PBXGroup section */

/* Begin PBXNativeTarget section */
Expand Down Expand Up @@ -12929,8 +12937,8 @@
isa = XCRemoteSwiftPackageReference;
repositoryURL = "https://github.com/duckduckgo/BrowserServicesKit";
requirement = {
kind = exactVersion;
version = 95.0.0;
branch = fcappelli/breakage_improvement_mac;
kind = branch;
};
};
AA06B6B52672AF8100F541C5 /* XCRemoteSwiftPackageReference "Sparkle" */ = {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
"kind" : "remoteSourceControl",
"location" : "https://github.com/duckduckgo/BrowserServicesKit",
"state" : {
"revision" : "ae9e9180f74d92c83fc3cc1d2fc23f4855fb361c",
"version" : "95.0.0"
"branch" : "fcappelli/breakage_improvement_mac",
"revision" : "12e535fcd2a8c3020c0718caccda3df629d1e02e"
}
},
{
Expand Down Expand Up @@ -77,8 +77,8 @@
"kind" : "remoteSourceControl",
"location" : "https://github.com/duckduckgo/privacy-dashboard",
"state" : {
"revision" : "38336a574e13090764ba09a6b877d15ee514e371",
"version" : "3.1.1"
"branch" : "12-14-feat_macos_allow_direct_navigation_to_breakage_form",
"revision" : "24b6260043a888827998124bc12104f4779e010f"
}
},
{
Expand Down
2 changes: 2 additions & 0 deletions DuckDuckGo/Application/AppDelegate.swift
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,8 @@ final class AppDelegate: NSObject, NSApplicationDelegate, FileDownloadManagerDel
private var emailCancellables = Set<AnyCancellable>()
let bookmarksManager = LocalBookmarkManager.shared

var privacyDashboardWindow: NSWindow?

private var didFinishLaunching = false

#if SPARKLE
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
{
"images" : [
{
"filename" : "Exclamation-16.pdf",
"idiom" : "universal"
}
],
"info" : {
"author" : "xcode",
"version" : 1
},
"properties" : {
"template-rendering-intent" : "template"
}
}
Binary file not shown.
4 changes: 3 additions & 1 deletion DuckDuckGo/Common/Localizables/UserText.swift
Original file line number Diff line number Diff line change
Expand Up @@ -432,6 +432,9 @@ struct UserText {
static let bookmarksManageBookmarks = NSLocalizedString("bookmarks.manage-bookmarks", value: "Manage Bookmarks", comment: "Menu item for opening the bookmarks management interface")
static let bookmarkImportedFromFolder = NSLocalizedString("bookmarks.imported.from.folder", value: "Imported from", comment: "Name of the folder the imported bookmarks are saved into")

static let reportBrokenSite = NSLocalizedString("report.broken.site", value: "Report Broken Site", comment: "Menu with feedback commands")
static let browserFeedback = NSLocalizedString("send.browser.feedback", value: "Send Browser Feedback", comment: "Menu with feedback commands")

static let zoom = NSLocalizedString("zoom", value: "Zoom", comment: "Menu with Zooming commands")

static let emailOptionsMenuItem = NSLocalizedString("email.optionsMenu", value: "Email Protection", comment: "Menu item email feature")
Expand Down Expand Up @@ -555,7 +558,6 @@ struct UserText {

static let sendFeedback = NSLocalizedString("preferences.about.send-feedback", value: "Send Feedback", comment: "Feedback button in the about preferences page")

static let feedbackBreakageDisclaimer = NSLocalizedString("feedback.breakage.disclaimer", value: "Reports sent to DuckDuckGo are 100% anonymous and only include your selection above, your optional message, the URL, a list of trackers we found on the site, the DuckDuckGo app version, and your macOS version.", comment: "Disclaimer in breakage form")
static let feedbackDisclaimer = NSLocalizedString("feedback.disclaimer", value: "Reports sent to DuckDuckGo are 100% anonymous and only include your message, the DuckDuckGo app version, and your macOS version.", comment: "Disclaimer in breakage form")

static let feedbackBugDescription = NSLocalizedString("feedback.bug.description", value: "Please describe the problem in as much detail as possible:", comment: "Label in the feedback form")
Expand Down
2 changes: 1 addition & 1 deletion DuckDuckGo/DataImport/ChromePreferences.swift
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ struct ChromePreferences: Decodable {
let profile: Profile

init(from data: Data) throws {
var decoder = JSONDecoder()
let decoder = JSONDecoder()
decoder.keyDecodingStrategy = .convertFromSnakeCase

self = try decoder.decode(Self.self, from: data)
Expand Down
Loading