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

Add basename function to sgml FuncMap #1104

Merged
merged 1 commit into from
Mar 9, 2023

Conversation

davidgamba-au
Copy link
Contributor

This function is necessary to allow custom backends to only use the basename of an image in cases where images are stored as attachments.

blockImageTmpl = `<image ><attachment filename="{{ basename .Src }}" /></image>`

@codecov
Copy link

codecov bot commented Feb 28, 2023

Codecov Report

Merging #1104 (3d22367) into master (f760dcb) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1104   +/-   ##
=======================================
  Coverage   77.59%   77.59%           
=======================================
  Files          89       89           
  Lines        8940     8941    +1     
=======================================
+ Hits         6937     6938    +1     
  Misses       1509     1509           
  Partials      494      494           

@xcoulon
Copy link
Member

xcoulon commented Feb 28, 2023

@davidgamba-au could you solve the conflicts on the PR, please?

@DavidGamba
Copy link
Contributor

@xcoulon merge conflicts fixed! Thanks for the quick review!

This function is necessary to allow custom backends to only use the
basename of an image in cases where images are stored as attachments.

blockImageTmpl = `<image ><attachment filename="{{ basename .Src }}" /></image>`
@DavidGamba
Copy link
Contributor

@xcoulon rebased the latest master again. PTAL when you have a chance. Thanks!

@xcoulon xcoulon merged commit 529d0d0 into bytesparadise:master Mar 9, 2023
@xcoulon
Copy link
Member

xcoulon commented Mar 9, 2023

thanks @DavidGamba!

@DavidGamba DavidGamba deleted the add-filepath-base branch March 9, 2023 16:40
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