-
Notifications
You must be signed in to change notification settings - Fork 307
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
[Customer Center] Add contact support button #4023
[Customer Center] Add contact support button #4023
Conversation
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
9df2454
to
d465d76
Compare
d0bbe8a
to
8fef6d1
Compare
d465d76
to
b675ce3
Compare
8fef6d1
to
b8bd53f
Compare
b675ce3
to
48485ae
Compare
b8bd53f
to
99ecb10
Compare
48485ae
to
1bae203
Compare
99ecb10
to
9efea65
Compare
1bae203
to
4000330
Compare
9efea65
to
4569a03
Compare
4000330
to
3fd6220
Compare
4569a03
to
647fd79
Compare
3fd6220
to
bd681e4
Compare
647fd79
to
76e80a6
Compare
bd681e4
to
58108a9
Compare
76e80a6
to
68385b1
Compare
58108a9
to
d00e4a7
Compare
68385b1
to
2af82b5
Compare
d00e4a7
to
417dc97
Compare
2af82b5
to
19fa733
Compare
417dc97
to
03a3906
Compare
19fa733
to
37cd710
Compare
03a3906
to
b9cb2f3
Compare
e60b2fe
to
183f250
Compare
f36de93
to
96e2c14
Compare
526570e
to
e38f240
Compare
84d3f0e
to
c0dc9d5
Compare
c0dc9d5
to
7dc9861
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
action: { | ||
if let supportInformation = self.supportInformation { | ||
let subject = self.localization.commonLocalizedString(for: .defaultSubject) | ||
let body = self.localization.commonLocalizedString(for: .defaultBody) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it might be good in future iterations to have this body include some info on the device/user, but we can do this in follow-up PRs 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes! I will do that this week, I think it will be super useful
RevenueCatUI/CustomerCenter/Data/CustomerCenterConfigTestData.swift
Outdated
Show resolved
Hide resolved
…swift Co-authored-by: Toni Rico <toni.rico.diez@revenuecat.com>
d98dc5f
to
802f957
Compare
2977e6b
into
integration/customer_support_workflow
Add contact support button to the
ManageSubscriptionsView