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

Edit clasp clone to include project names #170

Merged
merged 2 commits into from May 16, 2018
Merged

Edit clasp clone to include project names #170

merged 2 commits into from May 16, 2018

Conversation

campionfellin
Copy link
Collaborator

Now clasp clone shows projectId - (projectName) so users can more easily pick the project they want to clone.

Would we rather have it as projectId - projectName or projectName - projectId?

Signed-off-by: campionfellin campionfellin@gmail.com

  • npm run test succeeds.
  • npm run lint succeeds.
  • Appropriate changes to README are included in PR.

…name for better user experience

Signed-off-by: campionfellin <campionfellin@gmail.com>
@coveralls
Copy link

coveralls commented May 15, 2018

Coverage Status

Coverage remained the same at 14.082% when pulling 13179d1 on campionfellin:clasp-clone into c2ea9e3 on google:master.

@grant
Copy link
Contributor

grant commented May 15, 2018

Let's specify the name this way:
name – id

Please give a screenshot or example in PRs if you can.

Signed-off-by: campionfellin <campionfellin@gmail.com>
@campionfellin
Copy link
Collaborator Author

campionfellin commented May 16, 2018

screen shot 2018-05-15 at 5 27 32 pm

Any way to ask the Apps Scripts team to implement Project delete? I don't see it here: https://developers.google.com/apps-script/api/reference/rest/

(As you can see I should probably clean up some of my scripts)

@grant
Copy link
Contributor

grant commented May 16, 2018

Apps Script projects are Google Drive files.

  1. script.google.com/create
  2. drive.google.com
    See the recently created file.

Just like drive.files.list we can also drive.files.delete.

@grant
Copy link
Contributor

grant commented May 16, 2018

PR looks good, thanks!

@grant grant merged commit 3ac4cb9 into google:master May 16, 2018
@campionfellin campionfellin deleted the clasp-clone branch May 16, 2018 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants