Skip to content

ui: allow docHelp override using config.json#4745

Merged
yadvr merged 4 commits intoapache:4.15from
shapeblue:add-dochelpmappings-config
Apr 1, 2021
Merged

ui: allow docHelp override using config.json#4745
yadvr merged 4 commits intoapache:4.15from
shapeblue:add-dochelpmappings-config

Conversation

@shwstppr
Copy link
Copy Markdown
Contributor

@shwstppr shwstppr commented Mar 3, 2021

Description

config.json can have a property named 'docHelpMappings' which can be used to override docHelp suffixes.
In config.json admin can add mappings as follows:

...
"docHelpMappings": {
  "virtual_machines.html": "some.html",
  "some_string": "override_string"
}

UI will use these mappings and will make appropriate replacements in the documentation links for different sections and forms.

Addresses #4731
Documentation PR: apache/cloudstack-documentation#199

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

Made changes in config.json

...
  "docHelpMappings": {
    "virtual_machines.html": "page1.html",
    "templates.html": "page2.html"
  }

dochelpmappings

@shwstppr shwstppr marked this pull request as draft March 3, 2021 07:55
@shwstppr shwstppr force-pushed the add-dochelpmappings-config branch from 4b86656 to ad99722 Compare March 3, 2021 08:02
@yadvr yadvr added this to the 4.15.1.0 milestone Mar 3, 2021
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Mar 3, 2021

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@rhtyd a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

config.json can have a property named 'docHelpMappings' which can be used to override docHelp suffixes.
In config.json admin can add mappings as follows:

"docHelpMappings": {
  "virtual_machine.html": "some.html",
  "some_string": "override_string"
}

UI will use these mappings and will make appropriate replacements in the documentation links for different sections and forms.

Addresses apache#4731

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/4745 (JID-3846)

@shwstppr shwstppr force-pushed the add-dochelpmappings-config branch from ad99722 to d9729bc Compare March 3, 2021 08:12
@shwstppr shwstppr marked this pull request as ready for review March 3, 2021 08:12
@shwstppr
Copy link
Copy Markdown
Contributor Author

shwstppr commented Mar 3, 2021

@blueorangutan ui

@shwstppr shwstppr linked an issue Mar 3, 2021 that may be closed by this pull request
@blueorangutan
Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/4745 (JID-3847)

@yadvr yadvr requested a review from utchoang March 6, 2021 09:39
Copy link
Copy Markdown

@utchoang utchoang left a comment

Choose a reason for hiding this comment

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

LGTM

yadvr and others added 3 commits March 11, 2021 13:10
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr
Copy link
Copy Markdown
Contributor Author

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress. [S]

@shwstppr
Copy link
Copy Markdown
Contributor Author

@rhtyd moved overriding call in router.js and ActionButton.vue. Working fine. Please check

@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/4745 (SL-JID-24)

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Mar 18, 2021

LGTM I need to test this manually before merging, are the upstream doc PR raised @shwstppr (to https://github.com/apache/cloudstack-documentation ?)

@shwstppr
Copy link
Copy Markdown
Contributor Author

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Mar 24, 2021

@blueorangutan ui

@blueorangutan
Copy link
Copy Markdown

@rhtyd a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@blueorangutan
Copy link
Copy Markdown

UI build: ✔️
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/4745 (SL-JID-56)

@yadvr yadvr merged commit 2513683 into apache:4.15 Apr 1, 2021
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Apr 1, 2021

Manually tested - LGTM

  "docHelpMappings": {
    "adminguide/virtual_machines.html": "conceptsandterminology/index.html"
  },

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.

UI: Create help doc mapping via config.json

5 participants