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 registration creation shorthands (#7383) #7698

Merged
merged 1 commit into from
Feb 27, 2020
Merged

Add registration creation shorthands (#7383) #7698

merged 1 commit into from
Feb 27, 2020

Conversation

Legioth
Copy link
Member

@Legioth Legioth commented Feb 27, 2020

(cherry picked from commit 7f803f9)


This change is Reviewable

@vaadin-bot
Copy link
Collaborator

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 Page.java#L538: Remove this use of "executeJavaScript"; it is deprecated. rule
  2. INFO Page.java#L288: Do not forget to remove this deprecated code someday. rule
  3. INFO VaadinService.java#L142: Do not forget to remove this deprecated code someday. rule

@pleku pleku self-requested a review February 27, 2020 11:58
@pleku pleku added this to External Reviews in OLD Vaadin Flow ongoing work (Vaadin 10+) via automation Feb 27, 2020
@pleku pleku added +0.1.0 and removed +1.0.0 labels Feb 27, 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 12 of 12 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 27, 2020
@pleku pleku merged commit 217f3b9 into 2.2 Feb 27, 2020
OLD Vaadin Flow ongoing work (Vaadin 10+) automation moved this from Reviewer approved to Done - pending release Feb 27, 2020
@pleku pleku deleted the pick_7f803f9 branch February 27, 2020 12:02
@mehdi-vaadin mehdi-vaadin added this to the 2.2.0.alpha15 milestone Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
OLD Vaadin Flow ongoing work (Vaadin ...
  
Done - pending release
Development

Successfully merging this pull request may close these issues.

None yet

4 participants