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.

Sign up for GitHub

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

Jump to bottom

Incorrect link to pre-defined device #1131

Closed
ajcasagrande opened this issue Jun 12, 2023 · 2 comments · Fixed by #1136 or #1135
Closed

Incorrect link to pre-defined device #1131

ajcasagrande opened this issue Jun 12, 2023 · 2 comments · Fixed by #1136 or #1135
Assignees
@lenny-goodell
Labels
bug Something isn't working
Milestone
Napa

Comments

@ajcasagrande
Copy link
Contributor

ajcasagrande commented Jun 12, 2023 •
edited
Loading

🐞 Bug Report

Description and Minimal Reproduction [REQUIRED]

edgex-docs/docs_src/getting-started/Ch-GettingStartedSDK-Go.md

Line 207 in fa9d97f

2. A pre-created device for the random number device is provided in this documentation. Download **[random-generator-device.yaml](random-generator-device.yaml)** and save the file to the `~/edgexfoundry/device-simple/cmd/device-simple/res/devices` folder.

This link is the same as the device profile link above:

edgex-docs/docs_src/getting-started/Ch-GettingStartedSDK-Go.md

Line 195 in fa9d97f

2. A pre-created device profile for the random number device is provided in this documentation. Download **[random-generator-device.yaml](random-generator-device.yaml)** and save the file to the `~/edgexfoundry/device-simple/cmd/device-simple/res/profiles` folder.

@ajcasagrande ajcasagrande added the bug Something isn't working label Jun 12, 2023
@lenny-goodell
Copy link
Member

random-generator-device.yaml should be renamed to be random-generator-profile.yaml
random-generator-device.json should be converted to YAML resulting in it being random-generator-device.yaml .

@lenny-goodell lenny-goodell modified the milestones: Minnesota, Napa Jun 12, 2023
@lenny-goodell
Copy link
Member

Changes need to be on both the main and minnesota branches

@lenny-goodell lenny-goodell mentioned this issue Jun 12, 2023
Closed
@lenny-goodell lenny-goodell self-assigned this Jun 12, 2023
lenny-goodell pushed a commit to lenny-goodell/edgex-docs that referenced this issue Jun 12, 2023
fix: Adjust profile and devices files used in Getting Started Device …
4b28a9f 
…SDK Go

fixes edgexfoundry#1131

Signed-off-by: Leonard Goodell <leonard.goodell@intel.com>
@lenny-goodell lenny-goodell mentioned this issue Jun 12, 2023
Merged
1 task
lenny-goodell pushed a commit to lenny-goodell/edgex-docs that referenced this issue Jun 12, 2023
fix: [main] Adjust profile and devices files used in Getting Started …
f678c3d 
…Device SDK

fixes edgexfoundry#1131

Signed-off-by: Leonard Goodell <leonard.goodell@intel.com>
@lenny-goodell lenny-goodell mentioned this issue Jun 12, 2023
Merged
1 task
lenny-goodell pushed a commit to lenny-goodell/edgex-docs that referenced this issue Jun 12, 2023
fix: Adjust profile and devices files used in Getting Started Device …
6ee0bf6 
…SDK Go

fixes edgexfoundry#1131

Signed-off-by: Leonard Goodell <leonard.goodell@intel.com>
@lenny-goodell lenny-goodell linked a pull request Jun 12, 2023 that will close this issue
fix: [minnesota] Adjust profile and devices files used in Getting Started Device SDK Go #1135
Merged
1 task
lenny-goodell pushed a commit that referenced this issue Jun 13, 2023
fix: Adjust profile and devices files used in Getting Started Device …
29280de 
…SDK Go (#1135)

fixes #1131

Signed-off-by: Leonard Goodell <leonard.goodell@intel.com>
lenny-goodell pushed a commit that referenced this issue Jun 13, 2023
fix: [main] Adjust profile and devices files used in Getting Started …
4ca31ed 
…Device SDK (#1136)

fixes #1131

Signed-off-by: Leonard Goodell <leonard.goodell@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lenny-goodell lenny-goodell

Labels
bug Something isn't working
Projects
None yet
Milestone
Napa
2 participants
@lenny-goodell @ajcasagrande