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

wk4 jan_epic: MTL Manual bug - broken S5 logo/image links #2467

Closed
lijenn opened this issue Jan 19, 2022 · 4 comments
Closed

wk4 jan_epic: MTL Manual bug - broken S5 logo/image links #2467

lijenn opened this issue Jan 19, 2022 · 4 comments
Assignees
Labels
bug These are problems associated with technologies that need to be fixed right away. Operations Value stream activities in support of VA healthcare operations.
Milestone

Comments

@lijenn
Copy link
Collaborator

lijenn commented Jan 19, 2022

Identified during John & David's prep s05 on 1/19:

Broken links for logos and images, overall formatting is off.
The chapter looks out of sync in the table of contents too.

https://lzim.github.io/mtl/6-mtl-live-session-05.html#DontClick
image

@lijenn lijenn added the bug These are problems associated with technologies that need to be fixed right away. label Jan 19, 2022
@lijenn lijenn added this to the jan_2022 milestone Jan 19, 2022
@lijenn lijenn self-assigned this Jan 19, 2022
@lijenn lijenn added the Operations Value stream activities in support of VA healthcare operations. label Jan 19, 2022
@jamesmrollins jamesmrollins self-assigned this Jan 24, 2022
@jamesmrollins
Copy link
Collaborator

Hi @lijenn - In noted this problem also when I was updating the Test repo on Thursday night. When I have a chance, I will look into this also.

@lijenn lijenn changed the title wk3 jan_epic: MTL Manual bug - broken S5 logo/image links wk4 jan_epic: MTL Manual bug - broken S5 logo/image links Jan 24, 2022
@lijenn
Copy link
Collaborator Author

lijenn commented Jan 24, 2022

@jamesmrollins I will be looking into this today, thank you!

@lijenn
Copy link
Collaborator Author

lijenn commented Jan 25, 2022

@jamesmrollins I noticed we didn't have a FindReplace.yml for the lzim/mtl gh-pages branch so I added one.
I noticed in bookdown, it likes its .png files to come from the raw source so I tried to FNR .png -> .png?raw=true but it failed with this error code at: https://github.com/lzim/mtl/actions/runs/1743318466
image

Maybe I missed editing something in the YML file for it work in the gh-pages branch here?
https://github.com/lzim/mtl/blob/fnr-png_bookdown/.github/workflows/FindReplace.yml

Will look more into this myself tmrw, thank you!

FYI @mnallajerla

@jamesmrollins
Copy link
Collaborator

Hi @lijenn - I am grasping at a few straws here . . .

Try removing this space

image

Try adding a wildcard to the argument

find: "*.png"
replace: "*.png?raw=true"

Try escaping the characters

From the jacobtomlinson gha-find-replace issues log

regular expression syntax not documented

Syntax explanation

golang docs on regular expressions

For example

find: ".png"
replace: ".png\?raw\=true"

Or some combination . . . hope this helped!

@jamesmrollins jamesmrollins removed their assignment Feb 17, 2022
@lzim lzim closed this as completed Apr 12, 2022
@lijenn lijenn moved this to closed in bug_tracker Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug These are problems associated with technologies that need to be fixed right away. Operations Value stream activities in support of VA healthcare operations.
Projects
Status: closed
Development

No branches or pull requests

3 participants