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

Add close error prompt feature #7467

Merged
merged 2 commits into from
Feb 3, 2020
Merged

Add close error prompt feature #7467

merged 2 commits into from
Feb 3, 2020

Conversation

caalador
Copy link
Contributor

@caalador caalador commented Feb 3, 2020

Add feature dropped during merge?
Fixes #7211


This change is Reviewable

Add feature dropped during merge?
Fixes #7211
@project-bot project-bot bot added this to External Reviews in OLD Vaadin Flow ongoing work (Vaadin 10+) Feb 3, 2020
pleku
pleku previously approved these changes Feb 3, 2020
Copy link
Contributor

@pleku pleku left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r1.
Reviewable status: :shipit: complete! all discussions resolved, 1 of 1 LGTMs obtained

OLD Vaadin Flow ongoing work (Vaadin 10+) automation moved this from External Reviews to Reviewer approved Feb 3, 2020
OLD Vaadin Flow ongoing work (Vaadin 10+) automation moved this from Reviewer approved to Changes Requested Feb 3, 2020
@caalador caalador moved this from Changes Requested to External Reviews in OLD Vaadin Flow ongoing work (Vaadin 10+) Feb 3, 2020
@pleku pleku moved this from External Reviews to Iteration Reviews in OLD Vaadin Flow ongoing work (Vaadin 10+) Feb 3, 2020
OLD Vaadin Flow ongoing work (Vaadin 10+) automation moved this from Iteration Reviews to Reviewer approved Feb 3, 2020
Copy link
Contributor

@pleku pleku left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 1 of 1 files at r2.
Reviewable status: :shipit: complete! all discussions resolved, 1 of 1 LGTMs obtained

Copy link
Contributor

@pleku pleku left a comment

Choose a reason for hiding this comment

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

Not sure if this should have had its own IT which could have prevented the regression ...

Reviewable status: :shipit: complete! all discussions resolved, 1 of 1 LGTMs obtained

Copy link
Contributor Author

@caalador caalador left a comment

Choose a reason for hiding this comment

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

An IT just has the problem of how do you get a webpack exception. The current JUnit test is sort of ok, even if a tad heavy on the magical configuration.

Reviewable status: :shipit: complete! all discussions resolved, 1 of 1 LGTMs obtained

@caalador caalador merged commit fb70ab5 into master Feb 3, 2020
OLD Vaadin Flow ongoing work (Vaadin 10+) automation moved this from Reviewer approved to Done - pending release Feb 3, 2020
@caalador caalador deleted the issues/7211_close_prompt branch February 3, 2020 13:17
@pleku pleku moved this from Done - pending release to Released in OLD Vaadin Flow ongoing work (Vaadin 10+) Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Webpack error dialog should close on click
3 participants