search

artsy / mobile

Mobile Team TODO
http://www.objc.io/issues/22-scale/artsy/
84 stars 11 forks source link

Incorrect links in the playbook.md file #92

Closed etolstoy closed 8 years ago

etolstoy commented 8 years ago

Hi!

Sorry for rushing into your task tracker. I've been reading your playbook for some inspiration (I want to do the same in our mobile team) and encountered two broken links.

The first one is Artsy/Mobile:

Current members of the Mobile Practice can be found on the Artsy/Mobile index of our GitHub repo.

The second one is artsy/potential:

Most of the general Artsy Developer information for onboarding is covered in artsy/potential.

And thanks for your work with the community!

ashfurrow commented 8 years ago

Yikes! Thanks for letting us know. The first link is just to https://github.com/artsy/mobile (I'll fix that).

The artsy/potential link has unfortunately been closed by the broader engineering team. A repository containing info on all our web services exposed some business logic that posed a business risk; after speaking with the lead web engineer, we concluded that having a single source of truth for Artsy engineers was important, and that a "partially closed" solution didn't really exist, so we had to close it. I think a note that the potential repo is private should be added (I'll do that too).

Never worry about opening an issue on this repo, we have it here and open so that developers can ask us questions and offer suggestions for our workflow. If there's anything else I can help with, let me know!

etolstoy commented 8 years ago

Thanks for the comment and quick fix! 👍