Skip to content
This repository was archived by the owner on Dec 10, 2019. It is now read-only.

Conversation

@jisantuc
Copy link
Contributor

@jisantuc jisantuc commented Aug 1, 2018

Overview

This PR points to a specific release of the API spec by default, instead of whatever happens to be on
master.

It also removes the locally stored spec to avoid confusing people (and for fun!).

Testing

  • ci
  • initialize a client --
>>> from rasterfoundry.api import API
>>> client = API(api_token='your api token here')
>>> #or
>>> client = API('your refresh token here')
  • if it succeeds, you're good

Closes #57

Copy link

@notthatbreezy notthatbreezy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can't delete the spec in the root of the repository until we communicate that current users need to upgrade the python client

@jisantuc jisantuc force-pushed the feature/js/point-to-permalink-for-spec branch from 1c4a174 to d5cf468 Compare August 9, 2018 14:21
@jisantuc jisantuc dismissed notthatbreezy’s stale review August 9, 2018 14:28

Ok, I revived it.

@jisantuc jisantuc merged commit b6ef8c1 into develop Aug 9, 2018
@jisantuc jisantuc deleted the feature/js/point-to-permalink-for-spec branch August 9, 2018 14:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants