Generated file name is obscure about drawio-export-action HOT 5 CLOSED

There are few important things you should know this to become clear to you:

1. Path passed with output parameter is relative to the path! That's not stated in documentation, neither it is obvious.
2. Considering my example, all of my files were be named: "[FILENAME]-Page-N.png", where:
   1. [FILENAME] is the original file name (without extension).
   2. Page is literally the word page (OR its translation to some language, in my case it was sometimes english sometimes russian).
   3. N is the page number. Regularily, for ordinary .drawio diagrams, it will be 1.

Finally I had to use the following steps combination in order to achieve my original goal:

- name: Render .drawio Files
  uses: rlespinasse/drawio-export-action@v2
  with:
      format: png
      path: assets
      output: .
      action-mode: all

# WARNING! This step implies all of the .drawio sources are ONE PAGE diagrams only!
- name: Rename Files
    run: for f in assets/*.png; do mv "$f" "$(echo "$f" | sed 's/-.\+-[0-9]\+//g')"; done

from drawio-export-action.

I would also ask the developer for a few improvements, if they will be considered valueable:

1. Update documentation to make clear output parameter is relative to path.
2. Set the default output value to "." (for simplicity reasons).
3. Add script, renaming output files, containing only one page and removing the -Page-\d+ name part.
   I could've helped you providing pull request if you find any of these proposals useful!

from drawio-export-action.

Thanks for the report.

1. The documentation needs some rewrites to make clear some behaviors built on top of the export feature for the Drawio Desktop app.
2. The default value for output will remain export to avoid breaking changes in this version. Will consider updating the default value in the next major version. The next documentation should make clearer how this works.
3. An option already exists for that: https://github.com/rlespinasse/drawio-export-action#remove-page-suffix, which can be considered to be set to true in the next major version.

from drawio-export-action.

Oh, great news about the option! Thanks a lot!

from drawio-export-action.

🎉 This issue has been resolved in version 2.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

from drawio-export-action.