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

Dynamically generate navigation #973

Merged
merged 55 commits into from
Nov 19, 2019

Conversation

maxmarkus
Copy link
Contributor

Currently we have a manual navigation structure stored in navigation-nodes.json, this file stays but contains only some static items that are not reflected by .md files.

  • .md files contain now a (valid) json object surrounded by <!-- meta meta -->
  • metaData.categoryPosition is the sort position of a whole category
  • metaData.position is the sorting inside a category

Be aware that navigation-generated.json will be overwritten by npm run mock:develop-plugins as well as npm run start/export, so those commands should not run in parallel.

zarkosimic and others added 30 commits September 18, 2019 13:05
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
# Conflicts:
#	website/docs/src/luigi-config/extended/navigation.js
…nakhromova/luigi into 853-Create-styles-for-documentation
…naKhromova/luigi into 853-Create-styles-for-documentation
…naKhromova/luigi into marynaKhromova-853-Create-styles-for-documentation
…nal-linking

# Conflicts:
#	website/docs/src/luigi-config/extended/navigation.js
#	website/docs/src/luigi-config/styles/index.scss
#	website/docs/src/routes/_layout.svelte
#	website/docs/src/unified-plugins/rehype-luigi-linkparser.js
…gation-json

# Conflicts:
#	website/docs/scripts/markdown.unified-preview.js
#	website/docs/src/client.js
#	website/docs/src/docs/general-settings.md
#	website/docs/src/routes/_layout.svelte
#	website/docs/src/services/markdown.service.js
@alexandra-simeonova alexandra-simeonova self-assigned this Nov 18, 2019
@marynaKhromova marynaKhromova self-assigned this Nov 18, 2019
Copy link
Contributor

@zarkosimic zarkosimic left a comment

Choose a reason for hiding this comment

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

Good job! 👍

Copy link
Contributor

@marynaKhromova marynaKhromova left a comment

Choose a reason for hiding this comment

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

good job! 👍

@pekura pekura merged commit 9908960 into SAP:master Nov 19, 2019
@maxmarkus maxmarkus deleted the 961-generate-navigation-json branch November 19, 2019 15:57
JohannesDoberer added a commit to JohannesDoberer/luigi that referenced this pull request Nov 20, 2019
* master:
  Dynamically generate navigation (SAP#973)
  Copy to clipboard for code blocks (SAP#963)
  add link on landing page to luigi fiddle (SAP#968)
JohannesDoberer added a commit to JohannesDoberer/luigi that referenced this pull request Nov 26, 2019
* master: (275 commits)
  update luigi version (SAP#985)
  FAQ Documentation (SAP#982)
  Docu links leftovers (SAP#974)
  Bugfix invalid initial root navnode (SAP#979)
  Leftover documentation fixes (SAP#978)
  Navigate with anchors (SAP#976)
  remove jesusreal from codeowners, thanks for your great work in this project (SAP#980)
  Dynamically generate navigation (SAP#973)
  Copy to clipboard for code blocks (SAP#963)
  add link on landing page to luigi fiddle (SAP#968)
  Custom markdown attributes (SAP#967)
  Luigi architecture page (SAP#969)
  Dynamic internal linking (SAP#960)
  Update sitemap.xml (SAP#964)
  Luigi client installation document (SAP#957)
  Cm fix (SAP#952)
  Create styles for documentation (SAP#958)
  Update Luigi core dependency to npm 6.13.0 (SAP#953)
  Luigi example react (SAP#944)
  link fix (SAP#950)
  ...

# Conflicts:
#	core/examples/luigi-sample-angular/src/assets/auth-mock/login-mock.html
stanleychh pushed a commit to stanleychh/luigi that referenced this pull request Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation documentation tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants