[wip] Fleshing out the data backend for PR creation

[shana]

• Covert collections of strings to objects in SampleData
• Update xaml file to reflect Sample View Model
• Confirm xaml file reflects sample data at compile time
• Bind sample data to View Model
• Update xaml file to reflect View Model
• Confirm view reflects sample data at run time
• Bind real data to View Model

[shana]

The initialization code is failing because this constructor is not marked [ImportingConstructor], so MEF can't create an instance of this. Every class that's exported to MEF (marked with [Export..]) has to either have an empty public constructor or a constructor marked with [ImportingConstructor], so that MEF knows which constructor to invoke when instantiating things.

[shana]

I figured out why this failed to run, I've added a comment about it. You can go ahead and revert the eddd987 commit, or better yet, reset history to the commit before it and force push so we don't have it in the history, it's not particularly useful.

[shana]

Needs [ImportingConstructor]

[paladique]

No luck with getting sample data to populate at runtime, @shana suggested a sample side project/experiment with a simple example to try to understand why this isn't successful.

[paladique]

Looks like the assignee list is populating with everyone from the whole org. GetAllForRepository grabs all possible users, so we might need to filter this out.

[shana]

Yeah, it should only be listing the actual repository members. Could you check which GitHub API would return that? It should be somewhere on the API reference page on the web.

[paladique]

It should only be listing the actual repository members.

Searched the API for this, I'm currently using Issue.Assignee.GetAllForRepository which is GET /repos/:owner/:repo/assignees and returns all possible assignees, seems at that point it's not repo specific since anyone in the org can be a collaborator.

There is Repository.GetAllContributors which is /repos/:owner/:repo/contributors which we could compare against and put all common contributors at the top, and would filter out non-collaborators.

[shana]

There is Repository.GetAllContributors which is /repos/:owner/:repo/contributors which we could compare against and put all common contributors at the top, and would filter out non-collaborators.

Yup, that's a good idea!

[paladique]

Development of the PR Creation is now here