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

default org bug - load default org on skafos auth #44

Merged
merged 12 commits into from
Sep 27, 2018

Conversation

eecaro
Copy link
Contributor

@eecaro eecaro commented Sep 26, 2018

⚠️ ⚠️ ⚠️ Requires PRs: https://github.com/MetisMachine/argus/pull/311 https://github.com/MetisMachine/apoth/pull/118 ⚠️ ⚠️ ⚠️

skafos auth command does not reset default organization

Changes

  • Upon skafos auth, get the user's default organization, write it to the defaults.json file, and load the default organization into the environment
  • Add Requests to get and set the default_organization
  • If the defaults.json file has been edited to an invalid or unauthorized org, require reauth
  • Refactored error handling for listing organizations and setting the default organization
  • Upon skafos init, do not create the files with empty tokens and ids in metis config file

@eecaro eecaro requested review from wess, kmussel and jeremytregunna and removed request for kmussel and wess September 26, 2018 15:26
@tylerhutcherson
Copy link

Looks good!

@jeremytregunna jeremytregunna merged commit fc5b0d0 into master Sep 27, 2018
@jeremytregunna jeremytregunna deleted the ct-default-org-bug branch September 27, 2018 20:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants