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

Remove PS Team's migrated content from DevHub #2757

Closed
2 tasks done
mitovskaol opened this issue Jun 10, 2022 · 4 comments
Closed
2 tasks done

Remove PS Team's migrated content from DevHub #2757

mitovskaol opened this issue Jun 10, 2022 · 4 comments
Assignees
Labels
decision Marks issues where decisions are documented documentation Marks tasks related to creating documentation *team/ app development* This label is to mark tasks and Sprint Goals for App Development workstream

Comments

@mitovskaol
Copy link
Contributor

mitovskaol commented Jun 10, 2022

Now that the new Tech Docs site (https://beta-docs.developer.gov.bc.ca) is live, we need to remove any of our duplicate content from DevHub (https://developer.gov.bc.ca) and direct visitors to the new site.

Use this spreadsheet to check if the content is owned by an external owner. External content should remain in DevHub index files until the current DevHub is replaced with a new product.

Scope: this ticket applies to the content owned by the Platform Services team only and does not include the removal of the externally owned content.

Additional context

Google Sheet of externally owned content in DevHub: https://docs.google.com/spreadsheets/d/1EVFGwxQgdHZPtS2KzdKbJlJqQVDQzomKIe1Y62VpqcI/edit#gid=804654598

DevHub repository: https://github.com/bcgov/devhub-app-web

This issue is concerned with the Gatsby project in the web folder of the repository. This app generates the static site that users see when they visit DevHub. The web folder includes a detailed README with instructions for local development. When creating your .env.production file from .env.production.example, please see @ty2k for environment variables.

When we refer to "removing migrated content", we want to remove content from the deployed DevHub site while adding placeholder text at the existing URLs. This is important - our users have documentation pages bookmarked, and they are indexed by search engines. If we were to remove a page entirely without providing placeholder content, links to that page will 404. We want to keep the same URL but change the underlying content to a placeholder indicating that the new content lives at Tech Docs.

DevHub hosts documents generated from its own repository, but also from external repositories owned by the Platform Services team.

If you are replacing an internal document within the project, you can remove the old content while updating the topicRegistry to point to a new placeholder file.

If you are replacing a document from an external repository, you should remove the reference to it from a given topicRegistry file, and also create a new internal document that will use the same URL.

URLs are case-sensitive and generated based on the title of the Markdown document (as determined by the first # / <h1> heading). Because we are generating URLs based on the document titles, it's necessary to create/replace documents for every page we are retiring. In other words, it's not enough to go into each topicRegistry file and update the file reference to point to a single placeholder page - we need one placeholder page for each document being retired, ensuring that the titles for the new placeholder page are identical to the page being replaced.

One quirk of DevHub URLs is that content pages tend to live at both /<page title> and also /<topic>/<page title>. Neither of these are declared canonical, and both use cases will have potentially been bookmarked by users or indexed by search engines. The /<page title> format of a page gets surfaced through search, while the /<topic>/<page title> format is navigated to through the "topic" landing page UI.

It would be best to create the new placeholder documents inside of the DevHub repo, as opposed to creating new dependencies on external repos.

Example external page: "12 Factor Apps"

Example internal page: "Integrating Algolia Search Engine Into the Devhub"

Definition of done

  • Update DevHub index files to remove references to the content owned by Platform Services Team repo.
  • Add a placeholder for the removed pages that indicates that the content has moved. The placeholder can be:
The content you are trying to access has moved. If it is technical content, look for it on [BC Gov Private Cloud Platform's Technical Documentation website](https://beta-docs.developer.gov.bc.ca/). For non-technical content, check out our [new BC Gov Private Cloud Platform website](https://cloud.gov.bc.ca/private-cloud).

If you can't find the information you are looking for on either of these sites, please email the Platform Services team (<a href="mailto:PlatformServicesTeam@gov.bc.ca">PlatformServicesTeam@gov.bc.ca</a>).
@mitovskaol mitovskaol added the *team/ app development* This label is to mark tasks and Sprint Goals for App Development workstream label Jun 10, 2022
@mitovskaol mitovskaol added decision Marks issues where decisions are documented documentation Marks tasks related to creating documentation labels Aug 4, 2022
@lukegonis lukegonis assigned Kolezhanchik and unassigned ty2k Aug 4, 2022
@ty2k
Copy link
Member

ty2k commented Aug 17, 2022

This ticket is currently blocked by broken builds in bcgov/devhub-app-web PR #1766.

@ty2k
Copy link
Member

ty2k commented Aug 20, 2022

With builds now succeeding, this ticket is no longer blocked, and we are able to change our topicRegistry JSON configuration files to reference our newly merged placeholder documents from bcgov/devhub-app-web PR #1766.

@ty2k
Copy link
Member

ty2k commented Sep 15, 2022

The pull request bcgov/devhub-app-web #1766 is where the individual page references are being switched to point to the new boilerplate content. This pull request was deployed to dev and I gave our internal chat channel notice on Friday, September 9 that this content migration effort was going live. Today, I deployed to test and prod with the intention of merging this PR as soon as possible, in order to start a new PR where I can deploy the "If you can't find..." text from the master branch.

@ty2k
Copy link
Member

ty2k commented Sep 15, 2022

In pull request bcgov/devhub-app-web #1773, I've deployed all of @Kolezhanchik's changes (including the full boilerplate text) to production, and I believe this can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
decision Marks issues where decisions are documented documentation Marks tasks related to creating documentation *team/ app development* This label is to mark tasks and Sprint Goals for App Development workstream
Projects
None yet
Development

No branches or pull requests

4 participants