Skip to content

Conversation

@crazy-max
Copy link
Member

@crazy-max crazy-max commented May 7, 2022

needs #14685

create a Jekyll plugin to fetch remote resources so they will be available as preview in Netlify when PR are created. also adds another plugin to update api toc.

@netlify
Copy link

netlify bot commented May 7, 2022

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 2120bd1
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/62851cd89f686e0009675134
😎 Deploy Preview https://deploy-preview-14688--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 May 7, 2022

@crazy-max
Copy link
Member Author

added another commit to replace update-api-toc script with a jekyll plugin so we can see results in preview too. Before:

image

After:

image

@crazy-max crazy-max force-pushed the fetch-remote-plugin branch 4 times, most recently from 39dde62 to bd1eac0 Compare May 8, 2022 21:39
@crazy-max crazy-max force-pushed the fetch-remote-plugin branch from bd1eac0 to 547871d Compare May 9, 2022 13:59
@crazy-max
Copy link
Member Author

crazy-max commented May 9, 2022

As discussed with @thaJeztah, moved the JSON config for the fetch_remote plugin to the _config.yml for better visibility.

@crazy-max crazy-max force-pushed the fetch-remote-plugin branch from 547871d to eaf46d8 Compare May 11, 2022 14:06
@crazy-max crazy-max force-pushed the fetch-remote-plugin branch from eaf46d8 to 694a390 Compare May 18, 2022 16:01
crazy-max added 2 commits May 18, 2022 18:20
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <crazy-max@users.noreply.github.com>
@crazy-max crazy-max force-pushed the fetch-remote-plugin branch from 694a390 to 2120bd1 Compare May 18, 2022 16:20
@crazy-max crazy-max marked this pull request as ready for review May 18, 2022 16:22
@crazy-max
Copy link
Member Author

crazy-max commented May 19, 2022

@thaJeztah As discussed, updated the config yml with full repo uri as well as simplified git ref.

Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM!

@thaJeztah
Copy link
Member

YOLO! Let's get this one merged!

@thaJeztah thaJeztah merged commit cd5103f into docker:master May 19, 2022
@crazy-max crazy-max deleted the fetch-remote-plugin branch May 19, 2022 08:15
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