Skip to content

fix: Fixed arg naming for location updating#223

Merged
frgfm merged 1 commit into
masterfrom
azimuth-client
Oct 12, 2022
Merged

fix: Fixed arg naming for location updating#223
frgfm merged 1 commit into
masterfrom
azimuth-client

Conversation

@frgfm

@frgfm frgfm commented Oct 12, 2022

Copy link
Copy Markdown
Member

This PR fixes #219 where the client modification wasn't made to align with the new table column naming.

Any feedback is welcome!

@frgfm frgfm added type: fix Something isn't working ext: client labels Oct 12, 2022
@frgfm frgfm added this to the 0.2.0 milestone Oct 12, 2022
@frgfm
frgfm requested a review from a team October 12, 2022 10:11
@frgfm frgfm self-assigned this Oct 12, 2022
@codecov

codecov Bot commented Oct 12, 2022

Copy link
Copy Markdown

Codecov Report

Merging #223 (c3391ae) into master (fa878b8) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #223      +/-   ##
==========================================
+ Coverage   94.07%   94.08%   +0.01%     
==========================================
  Files          47       47              
  Lines        1282     1285       +3     
==========================================
+ Hits         1206     1209       +3     
  Misses         76       76              
Flag Coverage Δ
unittests 94.08% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/app/api/schemas/alerts.py 100.00% <0.00%> (ø)
src/app/api/routes/alerts.py 98.24% <0.00%> (+0.06%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@frgfm
frgfm merged commit 3618338 into master Oct 12, 2022
@frgfm
frgfm deleted the azimuth-client branch October 12, 2022 10:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ext: client type: fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant