Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to new dashboard #477

Merged
merged 5 commits into from
Aug 28, 2020
Merged

Switch to new dashboard #477

merged 5 commits into from
Aug 28, 2020

Conversation

medikoo
Copy link
Contributor

@medikoo medikoo commented Aug 20, 2020

Note: Logging in to new dashboard depends on @serverless/platform-client which in turn doesn't support Node.js v6. Therefore this cannot be made without introducing a breaking change (at least in context of login/logout functionality)

Switch from dashboard.serverless.com to app.serverless.com

Closes #474
Fixes #482

Additionally:

  • Removed enforced process.exit calls, as they can be error-prone
  • Removed login.test.js as it tested characteristics of implementation and not the desired outcome of implementation. Having that was a real obstacle in refactor

@medikoo medikoo added the enhancement New feature or request label Aug 20, 2020
@medikoo medikoo self-assigned this Aug 20, 2020
@codecov
Copy link

codecov bot commented Aug 20, 2020

Codecov Report

Merging #477 into v4 will decrease coverage by 0.51%.
The diff coverage is 39.39%.

Impacted file tree graph

@@            Coverage Diff             @@
##               v4     #477      +/-   ##
==========================================
- Coverage   53.86%   53.35%   -0.52%     
==========================================
  Files         107      106       -1     
  Lines        3145     3149       +4     
==========================================
- Hits         1694     1680      -14     
- Misses       1451     1469      +18     
Impacted Files Coverage Δ
index.js 1.35% <ø> (-0.45%) ⬇️
lib/studio.js 15.31% <0.00%> (-0.14%) ⬇️
lib/logout.js 25.00% <14.28%> (-15.00%) ⬇️
lib/login.js 22.22% <23.07%> (-77.78%) ⬇️
lib/plugin.js 85.48% <50.00%> (+0.07%) ⬆️
lib/dashboard.js 100.00% <100.00%> (ø)
lib/deployment/save.js 100.00% <100.00%> (ø)
lib/errorHandler.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 870a7f4...603e997. Read the comment docs.

@medikoo medikoo added the breaking Braking change (require major bump) label Aug 20, 2020
lib/login.js Outdated
@@ -1,24 +1,47 @@
'use strict';

const { login } = require('@serverless/platform-sdk');
const { ServerlessSDK } = require('@serverless/platform-client');
const { urls, readConfigFile, writeConfigFile, openBrowser } = require('@serverless/platform-sdk');
Copy link
Contributor Author

@medikoo medikoo Aug 24, 2020

Choose a reason for hiding this comment

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

Instead of openBrowser, we should rely on open dependency directly. So we also solve https://github.com/serverless/serverless/issues/7511

They test internal implementation characteristics and not desired outcome, which defeats tests purpose and turns them into an obstacle when refactoring.
@medikoo medikoo force-pushed the 0820-switch-to-new-dashboard branch from 76b0042 to bb95b8b Compare August 27, 2020 14:45
@medikoo medikoo changed the base branch from master to v4 August 27, 2020 14:46
@medikoo medikoo removed the breaking Braking change (require major bump) label Aug 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Login and link to new dashbaord (app.serverless.com)
1 participant