Skip to content
This repository has been archived by the owner on Jun 21, 2020. It is now read-only.

Development #61

Merged
merged 194 commits into from
Aug 9, 2017
Merged

Development #61

merged 194 commits into from
Aug 9, 2017

Conversation

jeffrey-opdam
Copy link
Contributor

Thank you for your pull request!
By completing this pull request, you agree to the Contributing License Agreement.

Fixes # .

Changes proposed in this pull request:

  • Minimal Viable Product

@ALM-Rangers/Team-Services-Bot

jeffrey-opdam and others added 30 commits April 5, 2017 09:37
… Added code analysis to bot project and fixed errors and warnings. Work incomplete.
- Removed the unnecassary try catch from messagesController
jeffrey-opdam and others added 24 commits July 2, 2017 22:01
* Temporarily removed account Uri lookup.

* Added more

* Added basics & basic tests for BuildsDialog.

* Added code for showing the BuildDefinitionCard

* Added acceptance test.
* Temporarily removed account Uri lookup.

* Added more

* Added basics & basic tests for BuildsDialog.

* Added code for showing the BuildDefinitionCard

* Added acceptance test.

* Fixed message
* Added QueueBuildAsync method and test

* Added logic for handling queing a build.

* Added acceptance test.

* Fixed some small bugs regarding Build Queue

* Fixed a warning
* Added method to get release definitions

* Added ReleasesDialog with unit tests for listing release definition.

* Added acceptance test for listing releases

* Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts.

* Added Create method to releasesdialog, added unittests, added acceptance tests.

* Renamed test name.

* Added GetReleaseAsyncTest
…ling

Proper error handling in AuthorizeController
* Updated guards in the extension method(s)

* Added Guards on the Cards.

* Added Guards to Bootstrap & Dialogs.
* Adhere to the naming

* Converted to VSTS
* Removed the common tests project.

* Removed ConfigurationProvider
Handling unknown commands.

Updated connect message.
@wpschaub wpschaub self-requested a review August 9, 2017 18:56
Copy link
Member

@wpschaub wpschaub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I picked up a small gremlin. Can you update the project name in the thirdpartynotice file?

https://github.com/ALM-Rangers/VSTS-Bot/blob/f8da72738a2b11fb44c3c87a1cea7ec7df2b757c/ThirdPartyNotices.txt#L5

@jeffrey-opdam
Copy link
Contributor Author

Done.

@wpschaub wpschaub merged commit fb04cbb into master Aug 9, 2017
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.

None yet

6 participants