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

Please add "amplify pull" #174

Closed
KoldBrewEd opened this issue Sep 17, 2018 · 8 comments
Closed

Please add "amplify pull" #174

KoldBrewEd opened this issue Sep 17, 2018 · 8 comments
Labels
feature-request Request a new feature

Comments

@KoldBrewEd
Copy link

The Mobile CLI had a "pull" command that allowed to sync changes on cloud resources to the local project. It'd be great to have the same capability on Amplify.

@UnleashedMind
Copy link
Contributor

Thanks for the feedbacks.

@kaustavghosh06
Copy link
Contributor

@awsed Currently we're using Clousdformation to manage the resources in an Amplify project and Cloudformation doesn't allow managing /updating/maintaining of resources created outside of Cloudformation (Its a cloudformation limitation).

So we're thinking and brainstorming of ways to make this possible but at the same time we don't come up with another set of templating mechanisms (similar to CFN) to just manage already provisioned resources.

Suggestions are welcome!

@KoldBrewEd
Copy link
Author

Thank you, @kaustavghosh06 . I thought that was the case, CFN indeed won't allow to sync manual changes. Maybe something more specific would be easier to implement...

My use case is related to AppSync. I'd love to deploy my GraphQL API, change and test in the console, add data sources and resolvers (for instance, there's no GraphQL transformer for Lambda so I have to add it manually) then do a "amplify api pull" and get the JSON file with all my new resolvers configs.

Then if I need to create a new API with the new config, I could basically copy the folder with the resolvers JSON files and deploy a similar API. Does it make sense?

@0xR
Copy link
Contributor

0xR commented Sep 21, 2018

It might be possible, but to me it seems to go against what amplify-cli is going for. It seems like the goal is to generate both the GraphQL schema as well as the resolvers based on graphql type definition with directives.

If you want to control over resolvers you need to add a transformer. This gives you a reusable way to generate code.

I wonder how long the awsmobile cli will still continue to work. Because that will still do what you want.

@KoldBrewEd
Copy link
Author

KoldBrewEd commented Sep 21, 2018

Don't agree an existing feature on the previous CLI goes against anything in the new CLI. Being able to pull changes means it's much easier to prototype and test in the console during development, giving more options to developers.

Unfortunately there aren't Transformers for all data sources yet. Yes, the Mobile CLI does what I want but I cannot use both in the same project.

@Cybernetixs
Copy link

Cybernetixs commented Oct 11, 2018

I also believe a "pull" feature would be helpful. I was doing some testing of an api's lambda functions in the console and making changes, so my local amplify function versions were no longer in sync. I spent all morning searching for that "pull/sync" feature and came across this thread.

@swyxio
Copy link
Contributor

swyxio commented Jan 6, 2020

amplify pull seems to be added as at v4.6.0 (in this undocumented PR #2742). suggest closing this issue.

@github-actions
Copy link

This issue has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels for those types of questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request a new feature
Projects
None yet
Development

No branches or pull requests

6 participants