Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@0xGabi
Copy link
Contributor

@0xGabi 0xGabi commented Jul 8, 2020

Fix an issue during the creation of the Repo entity. When there is no artifact metadata on the subgraph available we try to manually fetch it using the contentUri attribute. With the changes made in the latest version (#121), we introduced a regression bug pointing to the wrong data: app.contentUri -> app.version.contentUri.

@codecov
Copy link

codecov bot commented Jul 8, 2020

Codecov Report

Merging #126 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #126   +/-   ##
=======================================
  Coverage   23.92%   23.92%           
=======================================
  Files          59       59           
  Lines        1049     1049           
  Branches      168      168           
=======================================
  Hits          251      251           
  Misses        798      798           
Flag Coverage Δ
#unittests 23.92% <100.00%> (ø)
Impacted Files Coverage Δ
packages/connect-thegraph/src/queries/index.ts 100.00% <ø> (ø)
packages/connect-thegraph/src/parsers/roles.ts 68.96% <100.00%> (ø)

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 e48b2da...cb16501. Read the comment docs.

@0xGabi 0xGabi requested a review from bpierre July 8, 2020 21:51
@0xGabi 0xGabi merged commit 41742a8 into master Jul 8, 2020
@0xGabi 0xGabi deleted the fix-org-connector branch July 8, 2020 22:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants