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

Open feedback form upon tapping "Send Feedback" in the About page #473

Merged
merged 1 commit into from
Mar 17, 2022

Conversation

ayoy
Copy link
Collaborator

@ayoy ayoy commented Mar 17, 2022

Task/Issue URL: https://app.asana.com/0/inbox/1201621708108816/1201974902498489/1201978617922020/f
Tech Design URL:
CC: @tomasstrba

Description:
I removed Asana and SurveyMonkey URLs and extracted the code to present feedback form into FeedbackPresenter.

Steps to test this PR:

  1. Go to Preferences -> About -> Send feedback
  2. Verify that it shows feedback form
  3. Open a new tab and navigate to any website
  4. Tap three dots -> Send feedback
  5. Verify that it shows feedback form

Testing checklist:

  • Test with Release configuration

Internal references:

Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM

@brindy brindy self-assigned this Mar 17, 2022
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

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

LGTM! nice wee clean up :)

@brindy brindy assigned ayoy and unassigned brindy Mar 17, 2022
@ayoy ayoy merged commit 457a4bb into develop Mar 17, 2022
@ayoy ayoy deleted the dominik/send-feedback branch March 17, 2022 13:15
samsymons added a commit that referenced this pull request Mar 25, 2022
# By Tomas Strba (3) and others
# Via GitHub
* develop:
  Testing Checklist Updates (#480)
  Xcode Clean Up (#479)
  External App Scheme permission (#419)
  Update dashboard to correct version (#477)
  Update BSK to fix percent encoding query parameters (#475)
  bump version 0.21.0
  add alternate for paste and match with style (#474)
  First reponder issues when switching tabs (#464)
  Pre-beta copy updates (#472)
  Improvements to data clearing (#467)
  Open feedback form upon tapping "Send Feedback" in the About page (#473)
  new tab tweaks (#471)

# Conflicts:
#	DuckDuckGo.xcodeproj/project.pbxproj
#	DuckDuckGo.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	DuckDuckGo/Secure Vault/View/SaveIdentityPopover.swift
#	DuckDuckGo/Secure Vault/View/SaveIdentityViewController.swift
#	DuckDuckGo/Secure Vault/View/SavePaymentMethodPopover.swift
#	DuckDuckGo/Secure Vault/View/SavePaymentMethodViewController.swift
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.

2 participants