search

Wizcorp / node-pivotal

NodeJS API library for PivotalTracker
44 stars 20 forks source link

Should work with multiple users. #26

Open YourDeveloperFriend opened 10 years ago

YourDeveloperFriend commented 10 years ago

I'm only just recently skimming through this as an option to playing with Pivotal Tracker's API, and I noticed that the process revolves around only one user using one Secret Key. If I want multiple users to use my dashboard, then I'd run into complications. Maybe it should be something along the lines of:

pivotalUser = pivotal.new(<api token>);
pivotalUser.command(data);

What do you think?

stelcheck commented 10 years ago

Sounds good to me; we're going to have a note somewhere to warn people upgrading, however.

Would you be able to make a PR for this? If so, the only thing I would like is that we do instantiation the following way:

var Pivotal = require('pivotal');
var myPivotal = new Pivotal('my token');
YourDeveloperFriend commented 10 years ago

Yeah, let me see what I can do. I'm only on the discovery phase of the Pivotal app that I'm thinking of building, though, so it may be awhile.

YourDeveloperFriend commented 10 years ago

I changed it. It wasn't too hard. I'm thinking about how to implement a test to check two simultaneous accounts at once, but I'm not sure how to fit that into the waterfall model. Any thoughts?