Skip to content

Update Dev Guide - Add Image#11432

Merged
offtherailz merged 2 commits intomasterfrom
11394_dev_guide
Sep 10, 2025
Merged

Update Dev Guide - Add Image#11432
offtherailz merged 2 commits intomasterfrom
11394_dev_guide

Conversation

@ElenaGallo
Copy link
Copy Markdown
Contributor

Description

Update broken image here

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x", remove the others)

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

Issue

What is the current behavior?

#

What is the new behavior?

Breaking change

Does this PR introduce a breaking change? (check one with "x", remove the other)

  • Yes, and I documented them in migration notes
  • No

Other useful information

@ElenaGallo ElenaGallo added this to the 2025.01.02 milestone Sep 5, 2025
@ElenaGallo ElenaGallo self-assigned this Sep 5, 2025
@ElenaGallo ElenaGallo linked an issue Sep 5, 2025 that may be closed by this pull request
2 tasks
@ElenaGallo ElenaGallo requested a review from tdipisa September 5, 2025 16:29
@tdipisa tdipisa self-assigned this Sep 5, 2025
@tdipisa tdipisa requested review from offtherailz and removed request for tdipisa September 10, 2025 09:51
@tdipisa tdipisa assigned offtherailz and unassigned tdipisa and ElenaGallo Sep 10, 2025
@offtherailz
Copy link
Copy Markdown
Member

We now support again mermaid charts in PDF.
Therefore I restored the original chart made in mermaid syntax, and deleted your images.

@offtherailz offtherailz enabled auto-merge (squash) September 10, 2025 12:40
Copy link
Copy Markdown
Member

@offtherailz offtherailz left a comment

Choose a reason for hiding this comment

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

I fixed it restoring original mermaid charts instead of using images.
PDF is produced with it too.

@offtherailz offtherailz self-requested a review September 10, 2025 12:41
@offtherailz offtherailz merged commit 004cb9b into master Sep 10, 2025
5 checks passed
@tdipisa tdipisa added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Sep 10, 2025
offtherailz pushed a commit to offtherailz/MapStore2 that referenced this pull request Sep 10, 2025
@offtherailz
Copy link
Copy Markdown
Member

I did also the backport @ElenaGallo .
#11452

@offtherailz offtherailz removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Sep 10, 2025
@offtherailz offtherailz changed the title Update Dev Guide - Add image Update Dev Guide - Fix broken images on keycloak sso login Sep 10, 2025
@offtherailz offtherailz changed the title Update Dev Guide - Fix broken images on keycloak sso login Update Dev Guide - Add Image Sep 10, 2025
offtherailz added a commit that referenced this pull request Sep 10, 2025
Co-authored-by: ElenaGallo <56537133+ElenaGallo@users.noreply.github.com>
@offtherailz offtherailz deleted the 11394_dev_guide branch November 10, 2025 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dev Guide - Update broken image

4 participants