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

chore(config): updating config API to use v1 endpoint #496

Merged
merged 2 commits into from May 30, 2023

Conversation

shivamkm07
Copy link
Contributor

Description

This PR updates configuration API implementation to use the v1 stable endpoint instead of alpha1 endpoint.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
@shivamkm07 shivamkm07 requested review from a team as code owners May 30, 2023 07:58
@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Merging #496 (715690b) into main (b1fb0c2) will decrease coverage by 0.59%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #496      +/-   ##
==========================================
- Coverage   36.72%   36.14%   -0.59%     
==========================================
  Files          82       82              
  Lines        9927     9927              
  Branches      371      371              
==========================================
- Hits         3646     3588      -58     
- Misses       6222     6280      +58     
  Partials       59       59              
Impacted Files Coverage Δ
.../implementation/Client/GRPCClient/configuration.ts 9.09% <0.00%> (ø)

... and 1 file with indirect coverage changes

@shubham1172 shubham1172 added this to the Dapr Node.js SDK - v3.1.0 milestone May 30, 2023
shubham1172
shubham1172 previously approved these changes May 30, 2023
@shubham1172 shubham1172 self-requested a review May 30, 2023 08:49
@shubham1172
Copy link
Member

Hi @shivamkm07, could you please check the e2e test, looks like it's failing

image

(It still reports passing due to some weird bug).

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
@shubham1172
Copy link
Member

Awesome
image

@XavierGeerinck XavierGeerinck added this pull request to the merge queue May 30, 2023
Merged via the queue into dapr:main with commit 1114591 May 30, 2023
6 of 8 checks passed
jriffs pushed a commit to jriffs/js-sdk that referenced this pull request Jun 7, 2023
* updating config API to use v1 endpoint

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

* using latest runtime rc

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

---------

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
Signed-off-by: jriffs <mayjriffs97@gmail.com>
@shubham1172 shubham1172 changed the title updating config API to use v1 endpoint chore(config): updating config API to use v1 endpoint Jun 8, 2023
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.

None yet

3 participants