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

Fixed issue with apm packages command #1834

Merged
merged 2 commits into from
Mar 4, 2021

Conversation

nestorbe
Copy link
Contributor

@nestorbe nestorbe commented Mar 4, 2021

🦅 Pull Request Description

This PR is related to #1824 and does get merged into the QA PR called issue-1050/update-tests

Rational

The packages object was returning name and version properties, before the fix the table was trying to fetch a repo property that wasn't being returned by the object and this was causing the error.

✔️ PR Todo

This command (aragon apm packages) is still failing on Rinkeby. This is another issue that has to be fixed by someone with access to the aragonpm.eth registry: #1824 (comment)

@CLAassistant
Copy link

CLAassistant commented Mar 4, 2021

CLA assistant check
All committers have signed the CLA.

@nestorbe nestorbe changed the base branch from master to issue-1050/update-tests March 4, 2021 16:43
@nestorbe nestorbe requested review from nivida and c0rv0s March 4, 2021 16:46
@nivida nivida merged commit 7c7e7bf into issue-1050/update-tests Mar 4, 2021
@nestorbe nestorbe removed the request for review from c0rv0s March 4, 2021 16:53
@nivida nivida deleted the fix-apm_packages_command branch March 4, 2021 16:53
@welcome
Copy link

welcome bot commented Mar 4, 2021

Congrats on merging your first pull request! Aragon is proud of you 🦅
Eagle gif

c0rv0s pushed a commit that referenced this pull request Mar 16, 2021
* Fixed issue with apm packages command
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.

aragon apm packages fails on: xDAI, Rinkeby, Devchain
3 participants