Skip to content

Commit

Permalink
- HelpAndSupportViewController: change wording and no longer support …
Browse files Browse the repository at this point in the history
…sending emails, just a feedback URL (for a survey)

- MoreSettingsSection: change wording
- VendorServices+App: cleanup code
- Branding+App: remove feedback email address and add the iOS app survey's URL as a default in exchange
  • Loading branch information
felix-schwarz committed Jul 18, 2024
1 parent ab03572 commit 2e95ed3
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 14 deletions.
10 changes: 5 additions & 5 deletions ownCloud/Client/HelpAndSupportViewController.swift
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ class HelpAndSupportViewController: CollectionViewController {

cssSelectors = [.modal]

navigationItem.title = "Help & Support".localized
navigationItem.title = "Help & Contact".localized
navigationItem.largeTitleDisplayMode = .always

navigationItem.rightBarButtonItem = UIBarButtonItem(systemItem: .close, primaryAction: UIAction(handler: { [weak self] _ in
Expand Down Expand Up @@ -86,14 +86,14 @@ class HelpAndSupportViewController: CollectionViewController {
.spacing(20)
])

if (VendorServices.shared.feedbackMail != nil) || (Branding.shared.feedbackURL != nil) {
if Branding.shared.feedbackURL != nil {
elements.append(contentsOf: [
.title("Send feedback".localized),
.title("Feedback".localized),

.text("If you don't need a response and just want to send us feedback, you can also send us a mail.".localized, style: .informal, cssSelectors: [.message]),
.text("If you have a moment to give us your feedback, please take our survey.".localized, style: .informal, cssSelectors: [.message]),
.spacing(5),

.button("Send feedback".localized, action: UIAction(handler: { [weak self] _ in
.button("Take survey".localized, action: UIAction(handler: { [weak self] _ in
if let self {
VendorServices.shared.sendFeedback(from: self)
}
Expand Down
5 changes: 3 additions & 2 deletions ownCloud/Resources/en.lproj/Localizable.strings
Original file line number Diff line number Diff line change
Expand Up @@ -289,6 +289,7 @@
"More for {{title}}" = "More for {{title}}";
"Documentation" = "Documentation";
"Send feedback" = "Send feedback";
"Take survey" = "Take survey";
"Recommend to a friend" = "Recommend to a friend";
"Privacy Policy" = "Privacy Policy";
"Terms Of Use" = "Terms Of Use";
Expand Down Expand Up @@ -320,7 +321,7 @@
"Version information were copied to the clipboard" = "Version information were copied to the clipboard";

/* Help and support */
"Help & Support" = "Help & Support";
"Help & Contact" = "Help & Contact";

"Find information, answers and solutions in the detailed documentation." = "Find information, answers and solutions in the detailed documentation.";
"View documentation" = "View documentation";
Expand All @@ -329,7 +330,7 @@
"File an issue" = "File an issue";
"Visit the forums" = "Visit the forums";

"If you don't need a response and just want to send us feedback, you can also send us a mail." = "If you don't need a response and just want to send us feedback, you can also send us a mail.";
"If you have a moment to give us your feedback, please take our survey." = "If you have a moment to give us your feedback, please take our survey.";

/* User Interface Settings */
"Theme" = "Theme";
Expand Down
2 changes: 1 addition & 1 deletion ownCloud/Settings/MoreSettingsSection.swift
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ class MoreSettingsSection: SettingsSection {
if let viewController = self?.viewController {
VendorServices.shared.showHelpAndSupportOptions(from: viewController)
}
}, title: "Help & Support".localized, accessoryType: .disclosureIndicator, identifier: "help-and-support")
}, title: "Help & Contact".localized, accessoryType: .disclosureIndicator, identifier: "help-and-contact")

recommendRow = StaticTableViewRow(rowWithAction: { [weak self] (_, _) in
if let viewController = self?.viewController {
Expand Down
7 changes: 2 additions & 5 deletions ownCloud/Tools/VendorServices+App.swift
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ extension VendorServices {
public func sendFeedback(from viewController: UIViewController) {
if let sendFeedbackURL = Branding.shared.feedbackURL {
UIApplication.shared.open(sendFeedbackURL, options: [:], completionHandler: nil)
} else {
} else if let feedbackMail {
var buildType = "release".localized

if self.isBetaBuild {
Expand All @@ -61,10 +61,7 @@ extension VendorServices {
appSuffix = "-EMM"
}

guard let feedbackEmail = self.feedbackMail else {
return
}
self.sendMail(to: feedbackEmail, subject: "\(self.appVersion) (\(self.appBuildNumber)) \(buildType) \(self.appName)\(appSuffix)", message: nil, from: viewController)
self.sendMail(to: feedbackMail, subject: "\(self.appVersion) (\(self.appBuildNumber)) \(buildType) \(self.appName)\(appSuffix)", message: nil, from: viewController)
}
}

Expand Down
2 changes: 1 addition & 1 deletion ownCloudAppShared/Branding/Branding+App.swift
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ extension Branding : BrandingInitialization {
.helpURL : "https://owncloud.com/docs-guides/",
.privacyURL : "https://owncloud.org/privacy-policy/",
.termsOfUseURL : "https://raw.githubusercontent.com/owncloud/ios-app/master/LICENSE",
.sendFeedbackAddress : "ios-app@owncloud.com",
.sendFeedbackURL : "https://owncloud.com/ios-app-feedback",
.canAddAccount : true,
.canEditAccount : true,
.enableReviewPrompt : false,
Expand Down

0 comments on commit 2e95ed3

Please sign in to comment.