Skip to content

[bugfix] Fix for is1499 #1516

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

Merged
merged 14 commits into from
May 26, 2020
Merged

Conversation

ignapas
Copy link
Contributor

@ignapas ignapas commented May 20, 2020

What do these changes do?

Fix the reported problem with login after registering a user.

BONUS: Corrected bugs with not closing the feedback dialog when clicking continue and the rotating Idle icon.

Related issue number

closes #1499

How to test

Just register a user and check that it is redirected to the dashboard.

Checklist

  • Runs in the swarm

@ignapas ignapas requested a review from sanderegg May 20, 2020 12:55
@codecov
Copy link

codecov bot commented May 20, 2020

Codecov Report

Merging #1516 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1516      +/-   ##
==========================================
- Coverage   70.89%   70.88%   -0.01%     
==========================================
  Files         249      249              
  Lines       10097    10097              
  Branches     1114     1114              
==========================================
- Hits         7158     7157       -1     
- Misses       2617     2618       +1     
  Partials      322      322              
Flag Coverage Δ
#integrationtests 57.74% <ø> (ø)
#unittests 62.90% <ø> (-0.01%) ⬇️
Impacted Files Coverage Δ
.../director/src/simcore_service_director/producer.py 65.07% <0.00%> (-0.26%) ⬇️

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 3e7e5e6...800a724. Read the comment docs.

@ignapas ignapas requested review from pcrespov and odeimaiz May 20, 2020 12:55
@ignapas ignapas linked an issue May 20, 2020 that may be closed by this pull request
Copy link
Member

@odeimaiz odeimaiz left a comment

Choose a reason for hiding this comment

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

Schön!

@odeimaiz
Copy link
Member

I will PR the fix for the e2e

@odeimaiz odeimaiz self-requested a review May 25, 2020 09:36
Copy link
Member

@odeimaiz odeimaiz left a comment

Choose a reason for hiding this comment

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

👍 Thanks for the Readme

Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

minor

.gitignore Outdated
@@ -149,3 +149,7 @@ prof/

# outputs from make
.stack-*.yml

Copy link
Member

Choose a reason for hiding this comment

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

MINOR: For non-generics, I prefer if we create a specific .gitignore ... e.g. in this case it would be in tests/e2e/.gitignore

Add the following configuration to your local ``launch.json``:
```json
{
"type": "node",
Copy link
Member

Choose a reason for hiding this comment

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

This is very nice but It would even be better if you add it in .vscode-template/lauch.json ;-)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

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

Checked front-end and fixes the issues I reported in #1499 ! Great! :-)

Copy link
Member

@sanderegg sanderegg left a comment

Choose a reason for hiding this comment

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

👍

@ignapas ignapas merged commit d70285c into ITISFoundation:master May 26, 2020
@ignapas ignapas deleted the is1499/login-enh branch July 6, 2020 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Front end shall not show login page if session is logged in
4 participants