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

Add a support form to the website #7929

Merged
merged 10 commits into from Mar 1, 2021
Merged

Add a support form to the website #7929

merged 10 commits into from Mar 1, 2021

Conversation

ericholscher
Copy link
Member

@ericholscher ericholscher commented Feb 18, 2021

This will allow us to capture user information on form submission,
verifying that users are who they say.
This will save a few round trips in our support process,
and also be more user-friendly.

Screenshot

Screen Shot 2021-02-17 at 5 26 04 PM

This will allow us to capture user information on form submission,
verifying that users are who they say.
This will save a few round trips in our support process,
and also be more user-friendly.
@ericholscher ericholscher requested a review from a team February 18, 2021 01:22
docs/support.rst Outdated
User Support
------------

If you need a specific request for your project or account,
like more resources, change of the project's slug or username.
Send an email to support@readthedocs.org.
We will get to you as soon as possible.
Please fill out the form at https://readthedocs.org/support/,
Copy link
Member Author

Choose a reason for hiding this comment

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

Should this be a tab display with links for both Community & Business?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, that's the pattern we have already. It's not the best, but keeping consistency is better.

Copy link
Member

Choose a reason for hiding this comment

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

Also, we have plenty of places where we say "contact as " --we probably want to replace them all with the link to this new page.

Copy link
Member Author

Choose a reason for hiding this comment

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

@humitos I wonder if it's possible to create a URL that checks for a session on .com & .org, and redirect the user accordingly? If there is no active session, then we just show a small page with a selection between them. This seems useful if it isn't impossible with cookies.

Copy link
Contributor

Choose a reason for hiding this comment

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

We've thrown around a few additional options here too. We could also be doing this client side with javascript and a query param or something -- ie ../en/latest/?source=commercial. JS could read this an do some version of a CSS class change or something, though i'm not familiar with sphinx-tabs and what option we have here.

Long term, I would love to have an example of session inspection on a third party site -- hitting an external site via JS and using api return in string replacements. I've always imagined this would mostly be a sphinx extension though.

Copy link
Member

@humitos humitos left a comment

Choose a reason for hiding this comment

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

This is good. I'd happy if we have the same in commercial as well.

readthedocs/templates/support.html Show resolved Hide resolved
docs/support.rst Outdated
User Support
------------

If you need a specific request for your project or account,
like more resources, change of the project's slug or username.
Send an email to support@readthedocs.org.
We will get to you as soon as possible.
Please fill out the form at https://readthedocs.org/support/,
Copy link
Member

Choose a reason for hiding this comment

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

Yeah, that's the pattern we have already. It's not the best, but keeping consistency is better.

readthedocs/templates/support/index.html Outdated Show resolved Hide resolved
readthedocs/templates/support/index.html Outdated Show resolved Hide resolved
readthedocs/urls.py Outdated Show resolved Hide resolved
docs/support.rst Outdated
User Support
------------

If you need a specific request for your project or account,
like more resources, change of the project's slug or username.
Send an email to support@readthedocs.org.
We will get to you as soon as possible.
Please fill out the form at https://readthedocs.org/support/,
Copy link
Member

Choose a reason for hiding this comment

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

Also, we have plenty of places where we say "contact as " --we probably want to replace them all with the link to this new page.

<p> Please rate the severity of this event. </p>
{% else %}
<p>This option is only enabled for <a href="{% url "gold_subscription" %}">Gold users</a>.</p>
{% endif %}
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be enabled for commercial plans too. Is there a good way to do that here and not require an overridden template?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yea, I want an abstraction that works for both .org & .com around this. I don't think we have one, and didn't know if it was in scope for this PR :)

I haven't ported this to commercial yet, was planning to do that today and figure out a plan. I assume we'll still want to override the template because the upsell is different.

Copy link
Contributor

Choose a reason for hiding this comment

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

Good point, yeah we might want some more information on this page in commercial.

I suppose we could also unconditionally ask for the severity level and maybe avoid overriding the template. We wouldn't need to really use this information for anything but data collection, except for upper tier plans where response time varies by severity levels.

@humitos
Copy link
Member

humitos commented Feb 23, 2021

Not super relevant, but we could update our Twitter account to point to this form as well instead of mention github:

Screenshot_2021-02-23_08-47-50

@ericholscher ericholscher merged commit be9728a into master Mar 1, 2021
@ericholscher ericholscher deleted the support-form branch March 1, 2021 22:35
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.

None yet

4 participants