Skip to content
This repository has been archived by the owner on Apr 6, 2022. It is now read-only.

Correctly handle request to close from client-side #91

Merged
merged 4 commits into from
Aug 6, 2018

Conversation

denis-anisimov
Copy link

@denis-anisimov denis-anisimov commented Aug 3, 2018

Fixes #80 , #68


This change is Reviewable

@denis-anisimov denis-anisimov changed the title WIP: Correctly handles request to close from client-side WIP: Correctly handle request to close from client-side Aug 3, 2018
Copy link
Author

@denis-anisimov denis-anisimov left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion

a discussion (no related file):
IT tests are required to make sure that JS from the server side is correctly scheduled


Copy link
Author

@denis-anisimov denis-anisimov left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion

a discussion (no related file):

Previously, denis-anisimov (Denis) wrote…

IT tests are required to make sure that JS from the server side is correctly scheduled

Not IT but unit tests for the server side


@denis-anisimov denis-anisimov changed the title WIP: Correctly handle request to close from client-side Correctly handle request to close from client-side Aug 3, 2018
@vaadin-bot
Copy link

SonarQube analysis reported 3 issues

Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

  1. MINOR Dialog.java#L161: Move this constructor to comply with Java Code Conventions. rule
  2. MINOR Dialog.java#L410: Remove this method to simply inherit it. rule
  3. MINOR Dialog.java#L423: Remove this method to simply inherit it. rule

Copy link
Contributor

@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.

:lgtm:

Reviewed 3 of 3 files at r1, 2 of 2 files at r2.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

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

3 participants