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

allow for registering devices when building app for internal distribution #340

Merged
merged 2 commits into from
Apr 19, 2021

Conversation

dsokal
Copy link
Contributor

@dsokal dsokal commented Apr 16, 2021

Checklist

Why

Currently, if the user is running an internal distribution build but they haven't registered any devices yet, we throw an error Run 'eas device:create' to register your devices first.
This PR makes it possible to register the devices without the need to run another command.

How

I reused the device registration logic from device:create.

Test Plan

Tested manually.

@dsokal dsokal changed the title allow for registering devices when build app for internal distribution allow for registering devices when building app for internal distribution Apr 16, 2021
@dsokal dsokal force-pushed the @dsokal/register-devices-when-running-build branch from 3ec4aad to 57687f9 Compare April 16, 2021 13:47
@github-actions
Copy link

github-actions bot commented Apr 16, 2021

Size Change: +40 B (0%)

Total Size: 32.9 MB

Filename Size Change
./packages/eas-cli/dist/eas-linux-x64.tar.gz 32.9 MB +40 B (0%)

compressed-size-action

Base automatically changed from @dsokal/refactor-profile-validation to main April 19, 2021 10:29
@dsokal dsokal force-pushed the @dsokal/register-devices-when-running-build branch from 57687f9 to c7d3a08 Compare April 19, 2021 10:33
@dsokal dsokal merged commit a1d9012 into main Apr 19, 2021
@dsokal dsokal deleted the @dsokal/register-devices-when-running-build branch April 19, 2021 10:39
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