Use view=simple for MR API

Closes #34 (closed)

@stanhu is there any particular reason we did view=simple instead of simple=true? It seems like we use simple=true for the Projects API. Just noticed that discrepancy today

@ClemMakesApps This came from the review in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/12766#note_34640370. The idea was that this allows for the possibility of different views if necessary.

Hmm, do we plan on changing the other APIs that use simple=true in the future @stanhu?

assigned to @jschatz1

@smcgivern What do you think here? Should we just make our API consistent?

GraphQL is all views

I think the projects API should also accept a view param, and if we added simple in 9.4, we should be safe to remove it. @rymai wdyt? My reasoning for view is what if we add an extra_detail view in future?

1. view={simple,default,extra_detail}
2. simple=true or extra_detail=true, and the intersection of those isn't defined

Sorry that this MR isn't the best place for discussion but I figured its better that we bring it up than to let it slip by

@smcgivern Sounds good to me!

approved this merge request

merged

changed milestone to %1.0