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 zendesk tag support #979

Merged
merged 2 commits into from
Jan 5, 2024
Merged

Add zendesk tag support #979

merged 2 commits into from
Jan 5, 2024

Conversation

dgarcia360
Copy link
Collaborator

@dgarcia360 dgarcia360 commented Jan 5, 2024

This PR lets projects specify custom Zendesk tags in their conf.py file as follows:

html_theme_options = {
    'zendesk_tag': 'gq6ltsh3nfex3cnwfy4aj9',
}

Different from the previous approach, the tag is added to the main index.html file that handles redirections.

How to test

  1. Run make multiversion.

  2. Open the _build/dirhtml/index.html file.

  3. Check for the presence of the Zendesk tag associated with this project:

    image

Copy link
Collaborator

@annastuchlik annastuchlik left a comment

Choose a reason for hiding this comment

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

LGTM

@dgarcia360 dgarcia360 merged commit 8fd93ad into master Jan 5, 2024
5 checks passed
dgarcia360 added a commit to dgarcia360/scylla-manager that referenced this pull request Jan 9, 2024
Related: scylladb/sphinx-scylladb-theme#979

Adds the Zendesk tag to index.html page

## How to test

1. Run `make multiversion`.
1. Open the `_build/dirhtml/index.html` file.
1. Check for the presence of the Zendesk tag associated with this project.
karol-kokoszka pushed a commit to scylladb/scylla-manager that referenced this pull request Jan 11, 2024
Related: scylladb/sphinx-scylladb-theme#979

Adds the Zendesk tag to index.html page

## How to test

1. Run `make multiversion`.
1. Open the `_build/dirhtml/index.html` file.
1. Check for the presence of the Zendesk tag associated with this project.
Michal-Leszczynski pushed a commit to scylladb/scylla-manager that referenced this pull request Jan 17, 2024
Related: scylladb/sphinx-scylladb-theme#979

Adds the Zendesk tag to index.html page

## How to test

1. Run `make multiversion`.
1. Open the `_build/dirhtml/index.html` file.
1. Check for the presence of the Zendesk tag associated with this project.

(cherry picked from commit d837629)
@tomer-sandler
Copy link

@annastuchlik @dgarcia360 , please note that the meta-tag for the Manager docs was changed last week and I mistakenly deleted the previous crawler with its meta tag. Below is the correct tag, can you please edit the existing PR or create a new PR for it? Thanks!

manager.docs.scylladb.com/stable/sitemap.xml

<meta name='zd-site-verification' content='7x6adcx8o6lc6ug5w6tfvi' />

Michal-Leszczynski pushed a commit to scylladb/scylla-manager that referenced this pull request Jan 19, 2024
Related: scylladb/sphinx-scylladb-theme#979

Adds the Zendesk tag to index.html page

## How to test

1. Run `make multiversion`.
1. Open the `_build/dirhtml/index.html` file.
1. Check for the presence of the Zendesk tag associated with this project.

(cherry picked from commit d837629)
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.

3 participants