Skip to content

BXMSDOC-6638: Update case docs for 7.9 #2892

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

Merged
merged 2 commits into from
Oct 6, 2020

Conversation

sterobin
Copy link
Contributor

@sterobin sterobin commented Oct 2, 2020

@krisv @gmunozfe @michelehaglund , ready for your review on the three case docs. See doc previews below. I updated outdated screenshots and nav instructions in the three case docs, based on the latest UI for 7.9, and also made various other improvements/tweaks/corrections along the way to these three docs and to other docs that contained the same issues. No other changes.

I replied to you in Jira that the following BAPLs. These were initially linked for these case docs, but actually do not have doc impact for reasons I explain in the Jira. Though @fjtirado might need to update the KIE Server REST API swagger source accordingly, as I explained in the Jira.

Related links:

@sterobin sterobin added the under review Label for PRs pending SME or peer review label Oct 2, 2020
@sterobin sterobin changed the title BXMSDOC-6638: Update case docs for 7.9 WIP: BXMSDOC-6638: Update case docs for 7.9 Oct 2, 2020
@sterobin
Copy link
Contributor Author

sterobin commented Oct 2, 2020

jenkins retest this please

@sterobin
Copy link
Contributor Author

sterobin commented Oct 2, 2020

jenkins retest

Copy link
Contributor

@michelehaglund michelehaglund left a comment

Choose a reason for hiding this comment

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

Changes look great. Thank you for catching the same issues throughout other docs @sterobin

@sterobin
Copy link
Contributor Author

sterobin commented Oct 5, 2020

ok to test

Copy link
Contributor

@gmunozfe gmunozfe left a comment

Choose a reason for hiding this comment

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

Looks good to me, well done @sterobin !

@sterobin sterobin force-pushed the BXMSDOC-6638-master branch from e3e26ff to c304750 Compare October 5, 2020 18:41
Copy link
Contributor

@krisv krisv left a comment

Choose a reason for hiding this comment

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

Is there an easy way to check the rendered output @sterobin ? The use of a slash at the beginning of a URL seems to be inconsistent, so wanted to maybe check the difference in the output?

@sterobin
Copy link
Contributor Author

sterobin commented Oct 6, 2020

@krisv Fixed all, including the step about adding a case project, per our convo in chat. Also, regarding the backslashes, the rendered links are listed above for you to see. The inconsistency is intentional. The backslsash stops the URL from rendering as a clickable/active link. So where I don't want it to be clickable, backslash. Where I've it would be helpful for the user to just click and go, no backslash. This is something we're trying to standardize a bit in our docs for improved usability.

@sterobin sterobin removed the under review Label for PRs pending SME or peer review label Oct 6, 2020
@sterobin sterobin changed the title WIP: BXMSDOC-6638: Update case docs for 7.9 BXMSDOC-6638: Update case docs for 7.9 Oct 6, 2020
@sterobin sterobin merged commit 5347d39 into apache:master Oct 6, 2020
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 7, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 7, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit to tkobayas/kie-docs that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
tkobayas pushed a commit that referenced this pull request May 8, 2025
* BXMSDOC-6638: Update case docs for 7.9

* SME tweaks to gsg
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants