Skip to content

feat: Add toolbar feedback buttons with Sentry feedback form integration#33

Draft
philprime wants to merge 3 commits into
mainfrom
feature/feedback-button
Draft

feat: Add toolbar feedback buttons with Sentry feedback form integration#33
philprime wants to merge 3 commits into
mainfrom
feature/feedback-button

Conversation

@philprime
Copy link
Copy Markdown
Member

@philprime philprime commented Sep 18, 2025

Summary

  • Add a feedback megaphone toolbar button to all 6 main screens (lists, list detail, link detail, link info, create list, create link)
  • Introduce FeedbackAction environment value so views call feedback.show() without importing SentrySwift directly
  • Adopt the new SentryFeedbackFormConfig API from sentry-cocoa for manual feedback form presentation
  • Remove the floating feedback widget (showWidget()) in favor of explicit toolbar buttons
  • Remove dead widget configuration from configureUserFeedback
  • Add localization strings for the feedback button (label, accessibility label/hint)

Test plan

  • UI tests for all 6 feedback buttons pass (make test-ui-ios-app, 7/7 green)
  • Each test navigates to the screen, taps the feedback button, and verifies the Sentry feedback form opens
  • Tests run with TESTING=1 (Sentry disabled, in-memory DB) so no real feedback is sent
  • make build succeeds

@philprime philprime force-pushed the feature/feedback-button branch from 00ae2ba to a81e603 Compare June 2, 2026 11:40
…d UI tests

Introduce a FeedbackAction environment value so views call feedback.show()
instead of importing SentrySwift directly. Remove the floating feedback widget
in favor of toolbar buttons. Adopt the new SentryFeedbackFormConfig API from
sentry-cocoa. Add UI tests verifying the feedback button exists and opens the
form on all six screens.
@philprime philprime changed the title feat: Add toolbar button to display user feedback UI feat: Add toolbar feedback buttons with Sentry feedback form integration Jun 2, 2026
@sentry
Copy link
Copy Markdown

sentry Bot commented Jun 2, 2026

📲 Install Builds

iOS

🔗 App Name App ID Version Configuration
Flinky com.techprimate.Flinky 1.1.4 (6) --

⚙️ flinky Build Distribution Settings

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.

1 participant