openshift-docs icon indicating copy to clipboard operation
openshift-docs copied to clipboard

PR for SRVLOGIC-209: Add the "Openshift Serverless Logic overview" assembly.

Open kaldesai opened this issue 1 year ago • 13 comments
trafficstars

Affected versions: serverless-docs-1.33

Tracking JIRA: https://issues.redhat.com/browse/SRVLOGIC-209

Doc preview: Openshift Serverless Logic overview

kaldesai avatar Apr 25 '24 09:04 kaldesai

@kaldesai: This pull request references SRVLOGIC-209 which is a valid jira issue.

In response to this:

Tracking JIRA: https://issues.redhat.com/browse/SRVLOGIC-209

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Apr 25 '24 09:04 openshift-ci-robot

🤖 Mon Jul 15 15:44:38 - Prow CI generated the docs preview:

https://75129--ocpdocs-pr.netlify.app/openshift-serverless/latest/about/serverless-logic-overview.html

ocpdocs-previewbot avatar Apr 25 '24 09:04 ocpdocs-previewbot

@kaldesai: This pull request references SRVLOGIC-209 which is a valid jira issue.

In response to this:

Tracking JIRA: https://issues.redhat.com/browse/SRVLOGIC-209

Doc preview: Serverless Logic overview

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Apr 29 '24 13:04 openshift-ci-robot

Commented here as I did not know about this PR - https://github.com/openshift/openshift-docs/pull/70393

Also, comparing to the docs we have upstream, it feels like reduction of information that is available to the user. For example, callbacks overview gives no overview about callbacks, I doubt user will be able to understand what a callback is and what to use it for. Wdyt?

domhanak avatar May 13 '24 06:05 domhanak

@wmedvede @domhanak @fjtirado @ricardozanini I have incorporated your suggestions in the drafts.

kaldesai avatar Jul 01 '24 09:07 kaldesai

/label peer-review-needed

kaldesai avatar Jul 01 '24 09:07 kaldesai

@kaldesai: This pull request references SRVLOGIC-209 which is a valid jira issue.

In response to this:

Tracking JIRA: https://issues.redhat.com/browse/SRVLOGIC-209

Doc preview: Openshift Serverless Logic overview

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Jul 01 '24 09:07 openshift-ci-robot

@kaldesai: This pull request references SRVLOGIC-209 which is a valid jira issue.

In response to this:

Affected versions: serverless-docs-1.33

Tracking JIRA: https://issues.redhat.com/browse/SRVLOGIC-209

Doc preview: Openshift Serverless Logic overview

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

openshift-ci-robot avatar Jul 01 '24 09:07 openshift-ci-robot

Also, I did mark this once in the doc, but all of the headings need anchor IDs - there are a lot (especially 2nd level) in the modules that are missing these anchor IDs. I think this is needed for the publishing process and URL navigation.

kcarmichael08 avatar Jul 01 '24 14:07 kcarmichael08

Also, I did mark this once in the doc, but all of the headings need anchor IDs - there are a lot (especially 2nd level) in the modules that are missing these anchor IDs. I think this is needed for the publishing process and URL navigation.

I have included the anchor IDs and the context variables. Thank you so much for capturing this.

kaldesai avatar Jul 02 '24 08:07 kaldesai

@wmedvede @domhanak @fjtirado @ricardozanini I have incorporated your suggestions in the drafts.

kaldesai avatar Jul 05 '24 04:07 kaldesai

/label merge-review-needed

kaldesai avatar Jul 05 '24 04:07 kaldesai

/lgtm

I understand that more approvals are needed prior to merge, so I will stop here.

sheriff-rh avatar Jul 08 '24 12:07 sheriff-rh

New changes are detected. LGTM label has been removed.

openshift-ci[bot] avatar Jul 08 '24 12:07 openshift-ci[bot]

@kaldesai: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

openshift-ci[bot] avatar Jul 15 '24 15:07 openshift-ci[bot]

/cherrypick serverless-docs-1.33

jeana-redhat avatar Jul 15 '24 16:07 jeana-redhat

@jeana-redhat: new pull request created: #78893

In response to this:

/cherrypick serverless-docs-1.33

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.