Merge pull request #281 from criske/issue253

#253 Contributor stripe dashboard opens in a new tab.

src/main/resources/public/js/getAndAddContracts.js

@@ -11,6 +11,9 @@
 * but this a good practice since "$" is not "reserved" to jQuery.
 * Now "$" is just a function argument name and not the jQuery
 * alias anymore. It could named "foo" fo ex, thus making a call like foo("#contracts") valid.
+* @todo #253:60min On frontend, in Contracts tab, after adding a new contract, we should add the new
+*  contract to contract table, instead of updating table by re-fetching all contracts again.
+*  Same logic should be applied to "markContractForRemove" and "restoreContract".
 */
 (function getAndAddContracts($, contractsService, usersService, confirmDialog){
 

src/main/resources/public/js/wallets.js

@@ -139,7 +139,6 @@ function getProjectWallets() {
  * @param owner Owner of the repo/project.
  * @param name Name of the repo.
  * @param type Type of the wallet (fake or stripe).
- * @todo #195#60min On frontend, enable wallet activation. Right now `activateWallet()` function is not used.
  */
 function activateWallet(owner, name, type) {
     if(type == 'stripe') {

src/main/resources/templates/contributor.html

@@ -384,7 +384,9 @@ <h4>Account ID: <span id="accountId"></span></h4>
                             data: form.serialize(),
                             success: function (payoutMethod) {
                                 console.log("Opening SCA Dashboard page: " + payoutMethod.stripeLoginLink)
-                                window.location.replace(payoutMethod.stripeLoginLink);
+                                $("<a href=" + payoutMethod.stripeLoginLink + " target='_blank'></a>")[0].click();
+                                $("#stripeDashboardButton").removeClass("disabled");
+                                $("#loadingStripeDashboardForm").hide();
                             },
                             error: function(jqXHR, error, errorThrown) {
                                 $("#stripeDashboardButton").removeClass("disabled");