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

feat(data): Implement GET /event/device/name/{name} V2 API #2847

Merged
merged 1 commit into from
Oct 26, 2020
Merged

feat(data): Implement GET /event/device/name/{name} V2 API #2847

merged 1 commit into from
Oct 26, 2020

Conversation

weichou1229
Copy link
Member

@weichou1229 weichou1229 commented Oct 23, 2020

Fix #2842

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/master/.github/Contributing.md.

What is the current behavior?

Issue Number: #2842

What is the new behavior?

Implement CoreData GET /event/deice/name/{name} API to query events by offset, limit and device name.

Does this PR introduce a breaking change?

  • Yes
  • No

New Imports

  • Yes
  • No

Specific Instructions

Are there any specific instructions or things that should be known prior to reviewing?

Other information

@codecov-io
Copy link

codecov-io commented Oct 23, 2020

Codecov Report

Merging #2847 into master will increase coverage by 0.89%.
The diff coverage is 49.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2847      +/-   ##
==========================================
+ Coverage   38.81%   39.71%   +0.89%     
==========================================
  Files         145      166      +21     
  Lines       12722    13636     +914     
==========================================
+ Hits         4938     5415     +477     
- Misses       7510     7894     +384     
- Partials      274      327      +53     
Impacted Files Coverage Δ
internal/core/command/init.go 0.00% <0.00%> (ø)
internal/core/command/main.go 0.00% <0.00%> (ø)
internal/core/data/init.go 0.00% <0.00%> (ø)
internal/core/data/main.go 0.00% <0.00%> (ø)
internal/core/data/router.go 3.48% <ø> (-0.01%) ⬇️
internal/core/metadata/init.go 0.00% <0.00%> (ø)
internal/core/metadata/main.go 0.00% <0.00%> (ø)
internal/core/metadata/operators/device/notify.go 83.33% <0.00%> (-12.67%) ⬇️
internal/pkg/v2/infrastructure/redis/client.go 0.00% <0.00%> (ø)
internal/pkg/v2/infrastructure/redis/device.go 0.00% <0.00%> (ø)
... and 199 more

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 2a86d86...4853fd3. Read the comment docs.

Copy link
Member

@lenny-goodell lenny-goodell left a comment

Choose a reason for hiding this comment

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

Please fix typo in commit and PR titles. deice => device

Copy link
Member

@lenny-goodell lenny-goodell left a comment

Choose a reason for hiding this comment

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

Code looks good. Just need to fix commit and PR title pre previous comment

@weichou1229 weichou1229 changed the title feat(data): Implement GET /event/deice/name/{name} V2 API feat(data): Implement GET /event/device/name/{name} V2 API Oct 25, 2020
@weichou1229
Copy link
Member Author

Modified.

@cloudxxx8
Copy link
Member

please rebase and resolve the conflicts

@weichou1229
Copy link
Member Author

Resolved conflicts and rebased.

@cloudxxx8
Copy link
Member

recheck

Implement CoreData GET /event/device/name/{name} API to query events by offset, limit and device name.

Fix #2842

Signed-off-by: weichou <weichou1229@gmail.com>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 1 Code Smell

No Coverage information No Coverage information
15.3% 15.3% Duplication

@weichou1229 weichou1229 requested a review from cloudxxx8 October 26, 2020 13:36
Copy link
Member

@cloudxxx8 cloudxxx8 left a comment

Choose a reason for hiding this comment

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

LGTM

@lenny-goodell lenny-goodell merged commit 151af97 into edgexfoundry:master Oct 26, 2020
@weichou1229 weichou1229 deleted the issue-2842 branch October 27, 2020 01:17
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.

[V2 data] Implement GET /event/device/name/{name} V2 API
4 participants