Skip to content

Conversation

@christianlupus
Copy link
Collaborator

This should create a central wrapper module to encapsulate the Axios calls for better readability.

This should help to implement #935.

@codecov
Copy link

codecov bot commented Jul 1, 2022

Codecov Report

Merging #1061 (380c043) into master (933c3ce) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1061   +/-   ##
=======================================
  Coverage   44.58%   44.58%           
=======================================
  Files          50       50           
  Lines        1965     1965           
=======================================
  Hits          876      876           
  Misses       1089     1089           
Flag Coverage Δ
integration 2.29% <ø> (ø)
migration 4.32% <ø> (ø)
unittests 37.96% <ø> (ø)

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

@github-actions
Copy link

github-actions bot commented Jul 1, 2022

Unit Test Results

     36 files       36 suites   8m 29s ⏱️
   227 tests    227 ✔️ 0 💤 0
2 724 runs  2 724 ✔️ 0 💤 0

Results for commit 380c043.

♻️ This comment has been updated with latest results.

@christianlupus christianlupus merged commit 19a3e56 into master Jul 1, 2022
@christianlupus christianlupus deleted the maintenance/js-api-wrapper branch July 1, 2022 11:07
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.

2 participants