divshot/divshot-api

Wrapper for the Divshot API

divshot-api's Issues

provide method to generate projects's architect url

Add method to only list user's apps from a new API endpoint

api.apps.list() hits the /apps endpoint and should return back user + organization apps (see #13). In the near future, I'm envisioning the following collection of methods for retrieving apps:

api.apps.list() - Returns apps for self + organizations.
api.user.self().apps.list() - Returns apps for self.
api.organizations.id().apps.list() - Returns apps for a specific organization.

Having this flexibility allows me to reduce my requests or speed things up, depending on my needs:

api.apps.list() for generating a billing summary without multiple requests for organization apps.
api.user.self().apps.list() for faster requests in the Dashboard main view.

Change subscription.updateCard argument `card` to `token`

app.subscription = {
  updateCard: function (token, callback) {
    var url = app.url() + '/subscription/card';

    return app.http.request(url, 'PUT', {
      form: {
        card: token
      }
    }, callback);
  }
};

Rename npm module to "divshot-api"

Document "rollbackTo" in docs

Don't include a default Client ID

I want us to switch to using multiple applications, even for internal use. For instance, the CLI should be a different app from the dashboard should be a different app from the builder. We should move away from including a default Client ID in the code at all.

Provide generic http method that has pre-baked defaults

Rather than having to do one-offs like:

      $http(
        url: divshot._api.host + '/apps/' + name + '/subscription'
        method: 'DELETE'
        headers: divshot._api.headers
      )

We should be able to do something like

divshot.http(url: "/apps/" + name + "/subscription", method: "DELETE")

no conflict mode

deleteAccount method doesn't send email address

I think email needs to be wrapped inside form.

specific environment's releases should be "list()" instead of "get()"

• update documentation

Set API wrapper to use new version of API

https://github.com/divshot/divshot-api/blob/master/lib/Divshot.js#L10

If options.session is true, do not run authentication pre-hooks

If I create a client with the session option set to true, I am implying that I will not be using a Bearer token when making calls to the API.

As such, any calls made to protected resources that might normally trigger an authentication pre-hook (don't entirely understand the circumstances of this, but it seems to be happening) should be disabled.

Error Scenario

Using the Angular library, I configure it to use session:

divshotProvider.configure({client_id: 'abc123', session: true})

Making a single call at this point, e.g. divshot.user.self() completes successfully. However, other calls being made trigger a POST request to /token with no credentials resulting in a 401 and a setToken call with no token to set.

Once this has happened, the Authorization header (which should be Session <CLIENT_ID>) is Bearer undefined.

Refactor apps.list() method to list user + organization apps

At the moment api.apps.list() only returns the current user's apps (data.self), although the API endpoint returns all of the organization apps as well. I need a method that provides access to all of these apps to reduce the number of requests I need to make.

Browser Auth

Add a browser-only auth module with functionality to be built out by me (or someone else). Let's set it up as a pull request that will complete when everything's done.

@scottcorgan if you can start the branch, that'd be great.