Skip to content

fix: sync tutorial screenshots, resolve indefinite article grammar, ensure consistent indentation for alerts#218

Merged
rpietzsch merged 1 commit intoeccenca:mainfrom
MaximilianWenzel:bugfix/graph-insights-screenshot-and-template-sync
Feb 27, 2026
Merged

fix: sync tutorial screenshots, resolve indefinite article grammar, ensure consistent indentation for alerts#218
rpietzsch merged 1 commit intoeccenca:mainfrom
MaximilianWenzel:bugfix/graph-insights-screenshot-and-template-sync

Conversation

@MaximilianWenzel
Copy link
Contributor

Handles three minor issues:

  • Documentation sync: Replaced 4 tutorial screenshots that displayed the deprecated, parked mysparql domain. They now correctly reflect the safe http://example.org/northwind/ namespace to ensure visual consistency and eliminate any risk of routing users to an untrusted domain.
  • Template grammar: Fixed a bug in the templating engine that hardcoded the indefinite article (a {{ relation }}). The logic now correctly evaluates to "an" when appropriate (e.g., "an object property" rather than "a object property").
  • Whitespace cleanup: Removed unnecessary 4-space indentations on empty lines within alert boxes.

Copy link
Contributor

@rpietzsch rpietzsch left a comment

Choose a reason for hiding this comment

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

thank you for the PR. looks good to me.

@rpietzsch rpietzsch merged commit 5850bc2 into eccenca:main Feb 27, 2026
1 check failed
@MaximilianWenzel MaximilianWenzel deleted the bugfix/graph-insights-screenshot-and-template-sync branch February 27, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants