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

fix: Update all doc links in app template to refer to version 2.2 #1105

Merged
merged 1 commit into from
Apr 28, 2022

Conversation

lenny-goodell
Copy link
Member

fixes #1104

Signed-off-by: Leonard Goodell [email protected]

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/app-functions-sdk-go/blob/main/.github/CONTRIBUTING.md

PR Checklist

Please check if your PR fulfills the following requirements:

  • I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
  • I am not introducing a new dependency (add notes below if you are)
  • I have added unit tests for the new feature or bug fix (if not, why?) N/A
  • I have fully tested (add details below) this the new feature or bug fix (if not, why?) N/A
  • I have opened a PR for the related docs change (if not, why?) N/A

Testing Instructions

N/A

New Dependency Instructions (If applicable)

@@ -2,7 +2,7 @@

This folder contains a buildable/runnable template for a new custom application service based on the Pre-Release 2.x release of the App Functions SDK.

> **Note**: If you only need to use the built-in pipeline functions, then it is advisable that you use `App Service Configurable` rather then create a new custom application service. See [here](https://docs.edgexfoundry.org/1.3/microservices/application/AppServiceConfigurable/) for more details on `App Service Configurable`
> **Note**: If you only need to use the built-in pipeline functions, then it is advisable that you use `App Service Configurable` rather then create a new custom application service. See [here](https://docs.edgexfoundry.org/2.2//microservices/application/AppServiceConfigurable/) for more details on `App Service Configurable`
Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra /:

Suggested change
> **Note**: If you only need to use the built-in pipeline functions, then it is advisable that you use `App Service Configurable` rather then create a new custom application service. See [here](https://docs.edgexfoundry.org/2.2//microservices/application/AppServiceConfigurable/) for more details on `App Service Configurable`
> **Note**: If you only need to use the built-in pipeline functions, then it is advisable that you use `App Service Configurable` rather then create a new custom application service. See [here](https://docs.edgexfoundry.org/2.2/microservices/application/AppServiceConfigurable/) for more details on `App Service Configurable`

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed. Thanks!

@@ -48,13 +48,13 @@ type Sample struct {
}

// LogEventDetails is example of processing an Event and passing the original Event to next function in the pipeline
// For more details on the Context API got here: https://docs.edgexfoundry.org/1.3/microservices/application/ContextAPI/
// For more details on the Context API got here: https://docs.edgexfoundry.org/2.2//microservices/application/ContextAPI/
Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra /

func (s *Sample) LogEventDetails(ctx interfaces.AppFunctionContext, data interface{}) (bool, interface{}) {
lc := ctx.LoggingClient()
lc.Debugf("LogEventDetails called in pipeline '%s'", ctx.PipelineId())

if data == nil {
// Go here for details on Error Handle: https://docs.edgexfoundry.org/1.3/microservices/application/ErrorHandling/
// Go here for details on Error Handle: https://docs.edgexfoundry.org/2.2//microservices/application/ErrorHandling/
Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra /

@@ -162,7 +162,7 @@ func (s *Sample) OutputXML(ctx interfaces.AppFunctionContext, data interface{})

// This sends the XML as a response. i.e. publish for MessageBus/MQTT triggers as configured or
// HTTP response to for the HTTP Trigger
// For more details on the SetResponseData() function go here: https://docs.edgexfoundry.org/1.3/microservices/application/ContextAPI/#complete
// For more details on the SetResponseData() function go here: https://docs.edgexfoundry.org/2.2//microservices/application/ContextAPI/#complete
Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra /

@@ -68,7 +68,7 @@ Timeout = "30s"
# if not this secion can be removed, but you must make sure EDGEX_SECURITY_SECRET_STORE is set to false
# Note is database is running in secure more and you have Store and Forward enable you will need to run this
# service in secure mode.
# For more deatils about SecretStore: https://docs.edgexfoundry.org/1.3/microservices/security/Ch-SecretStore/
# For more deatils about SecretStore: https://docs.edgexfoundry.org/2.2//microservices/security/Ch-SecretStore/
Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra /

@@ -157,15 +157,15 @@ Type="edgex-messagebus"

# TODO: Add custom settings needed by your app service or remove if you don't have any settings.
# This can be any Key/Value pair you need.
# For more details see: https://docs.edgexfoundry.org/1.3/microservices/application/GeneralAppServiceConfig/#application-settings
# For more details see: https://docs.edgexfoundry.org/2.2//microservices/application/GeneralAppServiceConfig/#application-settings
Copy link
Contributor

Choose a reason for hiding this comment

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

remove extra /

@codecov-commenter
Copy link

Codecov Report

Merging #1105 (2e65055) into main (8ff173f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1105   +/-   ##
=======================================
  Coverage   68.49%   68.49%           
=======================================
  Files          37       37           
  Lines        3034     3034           
=======================================
  Hits         2078     2078           
  Misses        827      827           
  Partials      129      129           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8ff173f...2e65055. Read the comment docs.

@lenny-goodell lenny-goodell merged commit 8d5a86b into edgexfoundry:main Apr 28, 2022
@lenny-goodell lenny-goodell deleted the fix-doc-links branch April 28, 2022 17:39
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.

[SDK] Update links in App Template to point to 2.2 Docs
3 participants