Previously, long owner names could wrap onto two lines in the bot
management modal, breaking the layout. This commit adds CSS to keep
names on a single line and abbreviate with an ellipsis if too long.
Fixes#23266.
These files were converted and moved to
starlight_help/src/content/docs, but were not removed due to an
incorrect rebase of d333ddb961 (#35621).
Signed-off-by: Anders Kaseorg <anders@zulip.com>
The list inside the code block should not have any Steps component
inside it, since that is reserved for our MDX, not the formatting guide
for the Zulip apps.
Due to a missing `not`, this incorrectly skipped checking fragments
for everything *other* than the /help documentation!
Signed-off-by: Anders Kaseorg <anders@zulip.com>
Commits d333ddb961 and
a766c092fc (#35621) broke this by
changing the base class from UnusedImagesLinterSpider to
BaseDocumentationSpider.
Signed-off-by: Anders Kaseorg <anders@zulip.com>
The `article` variable being passed to `get_path` can also be a relative
path, which has it's security implications. By using `secure_filename`,
we mitigate that risk. We don't need to check if `/` exists in article
anymore since `secure_filename` will do so on it's own.
The sidebar index was an includes file in our previous help center since
we were using it to programmatically in documentation.py. Since that is
no longer the case, there is no reason for this file to be in includes.
A side-effect of us doing this is that now the overview section on the
right for the index page will be generated correctly with us avoiding
the issue in #35127.
Fixes#35533.
We could not convert them in the source markdown files since removing
the preceding line for tips and tables was breaking our custom markdown
implementation.
We have a copy of help center with relative links disabled which is
reserved for root domains without an organisation on the root domain.
Ideally, we should have some logic to determine whether we are on such
a root domain or not. For practical short term purposes, since this
type of documentation is mainly useful for zulip.com, we add an
exception for zulip.com.
Fixes#35131.
Most of the nginx config is copied from
https://docs.astro.build/en/recipes/docker/#nginx.
We ideally should be generating two builds for Zulip cloud and serve a
different one depending on whether the subdomain is same as the root
domain or not. We can look into this as an immediate followup while this
commit helps things get ready for a test deploy on CZO.
We add a step to build help center and then test the broken links as we
used to before removing the test temporarily.
This commit focuses on just adding back the broken link checks for the
help center. We skip the fragment check since that is in-built in
starlight and starlight tests account for that already. For the image
check we can add it back in a followup issue.
This commit removes the current help center markdown files and any logic
that was used to host those files at help/.
We also remove a bunch of tests, we should the equivalent of those tests
for the new help center. Issues to track: #35649, #35647. These issues
track adding back tests for redirects and broken links.
We had a symlink from templates/zerver/integrations/email.md pointing to
help/using-zulip-via-email.md. We can no longer have that symlink since
the latter has been converted to an MDX file. We have deleted the
symlink and put a markdown file in it's place. Both the files have
comments to edit the other in case of changes.
This commit also makes changes in astro config, astro component paths
and other places to move the starlight help center docs base path from
/starlight_help to /help.
The change to rename /starlight_help/ to /help/ in MDX files is done in
the next commit. If we squash these commits, this line should be
removed.
`./tools/build-help-center` no longer does the conversion step.
We also remove some dead code related to the old help center in
documentation.py.
We have not yet removed the help files. We can use this commit as a
conversion commit since a lot more help changes will be merged before
this PR gets merged. We will need to rebase on top of main, run
conversion step in this commit, push those changes and then merge the
PR.
NOTE: This commit temporarily breaks the astro build since we have
renamed the path in the MDX files from starlight_help/ to help/ early.
That should be fine since astro build will not break any tests right
now.
We should squash this commit with the next commit removing help center
files when merging into main and remove this NOTE.
Fixes#35649.
We do not serve the astro build via Django and thus we have to move the
help center redirects. We get a warning by pagefind on all these
redirects that they will be ignored in the search results as they don't
have an HTML body. We can ignore this warning till the cutover and
create a followup issue to solve that warning if this commit goes
through main.
When we copied over the logic for the footer from the previous help
center, I forgot to remove the double curly braces which were getting
rendered as is in astro.
Earlier, we removed topic display name for empty topic operand when
query was for topic autocomplete. But we should make sure that we
only do this for the last search term in the suggestion which
corresponds to the typed query and not any previously selected
entered search term. Also recently we removed subset suggestions,
so can safely check it for last search term for all suggestions.
This commit fixes this by checking if the topic suggestion is the
last search_term in the suggestion/search_terms.
Co-authored-by: Evy Kassirer <evy@zulip.com>
The import code wasn't setting subscriber_count at all, resulting in a
value of 0 when dealing with imports from 3rd party apps.
We run this unconditionally, also for imports from Zulip, since this
ensures we won't inherit incorrect values, if the data we're import has
them - e.g. due to some bugs that affected the server the data came from.
