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

CCDM: make IndexHtmlHandler serve any request, execute it the last #7508

Merged
merged 2 commits into from
Feb 7, 2020

Conversation

manolo
Copy link
Member

@manolo manolo commented Feb 5, 2020

This change is Reviewable

@project-bot project-bot bot added this to External Reviews in OLD Vaadin Flow ongoing work (Vaadin 10+) Feb 5, 2020
@caalador caalador removed this from External Reviews in OLD Vaadin Flow ongoing work (Vaadin 10+) Feb 6, 2020
@manolo manolo force-pushed the mcm/master/fix-dot-routes branch 2 times, most recently from d62f3b2 to f83c7b9 Compare February 6, 2020 06:44
@manolo manolo self-assigned this Feb 6, 2020
@manolo manolo force-pushed the mcm/master/fix-dot-routes branch 2 times, most recently from dd4877c to edff250 Compare February 6, 2020 10:46
Copy link
Contributor

@haijian-vaadin haijian-vaadin left a comment

Choose a reason for hiding this comment

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

Reviewed 7 of 7 files at r1.
Reviewable status: all discussions resolved, 0 of 1 LGTMs obtained (waiting on @ptdatkhtn)

Copy link
Contributor

@haijian-vaadin haijian-vaadin left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! all discussions resolved, 1 of 1 LGTMs obtained (waiting on @ptdatkhtn)

@haijian-vaadin haijian-vaadin merged commit e77492b into master Feb 7, 2020
@haijian-vaadin haijian-vaadin deleted the mcm/master/fix-dot-routes branch February 7, 2020 08:38
jm-ferreira pushed a commit that referenced this pull request Feb 7, 2020
…7508)

* CCDM: make IndexHtmlHandler serve any request, but put it the la position

* CCDM: allow configuring request handled by IndexHtmlHandler
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.

None yet

3 participants