Skip to content

Conversation

@crazy-max
Copy link
Member

@crazy-max crazy-max commented Oct 27, 2022

follow-up #15994 (comment)

Fix last_modified_at plugin that overrides last_modified_at frontmatter value for remote resources. And use data files for commands reference instead of stub files to compute the correct last modification date.

Also adds DOCS_ENFORCE_GIT_LOG_HISTORY option to enforce git log history for local dev for testing purpose.

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
…tter

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
…ad of stub file

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@netlify
Copy link

netlify bot commented Oct 27, 2022

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit a997dc9
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/635a7600c6830f000887e841
😎 Deploy Preview https://deploy-preview-15998--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@crazy-max
Copy link
Member Author

crazy-max commented Oct 27, 2022

@usha-mandya @dvdksn Not related to this PR but link to Hub pricing is broken:

- ./_site/subscription/faq/index.html
  *  https://
hub.docker.com/billing/plan/update is an invalid URL (line 255)
     <a href="https://
hub.docker.com/billing/plan/update" target="blank" rel="noopener" class="">the billing</a>
  *  linking to internal hash #pricing-plans that does not exist (line 210)
     <a href="/subscription/#pricing-plans" target="blank" rel="noopener" class="">billing</a>

Fixed it, see last commit

Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Copy link
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

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

LGMT, ptal @jerae-duffin

@dvdksn dvdksn merged commit 4544982 into docker:main Oct 31, 2022
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.

2 participants