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

Update help URLs and navigation text for Pendo #6505

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

a13m
Copy link
Contributor

@a13m a13m commented May 6, 2024

Proposed changes

  • Updated support and knowledgebase URLs
  • Update API key retrieval instructions to match current product navigation

Merge timing

  • ASAP once approved

@a13m a13m requested a review from a team as a code owner May 6, 2024 16:56
@a13m a13m requested review from cmastr and removed request for a team May 6, 2024 16:56
@cmastr cmastr requested a review from tcgilbert May 6, 2024 17:30
Site settings > Basic Information > API key. If your account has not fully
been set up yet, you can see the API key inside of the code snippet on the
To find your API key, have a Pendo admin log into Pendo, navigate to
Settings > Subscription Settings > Applications, open the relevant app,
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe these updates have to happen in another repo. Is that right @tcgilbert?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cmastr I can pull this change out of the PR if everything else is okay. My primary goal is to update our knowledgebase URLs.

Copy link
Contributor

Choose a reason for hiding this comment

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

@cmastr correct. this will simply get overwritten on next rebuild. So I would remove any changes to this file.

@a13m I updated the text for this setting on our back end. next time the docs site rebuilds this will be updated.
Screenshot 2024-05-08 at 11 45 21 AM

Copy link
Contributor

Choose a reason for hiding this comment

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

@a13m if you'll pull this change out, then I'll approve and merge. Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@cmastr done! Thank you.

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

3 participants