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

Change path rewriting to use req.accepts in serve mode #253

Merged
merged 2 commits into from
Mar 7, 2019

Conversation

matt-gadd
Copy link
Contributor

Type: bug
The following has been addressed in the PR:

  • All code has been formatted with prettier

Description:
The previous fix didn't actually work

@codecov
Copy link

codecov bot commented Mar 7, 2019

Codecov Report

Merging #253 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #253   +/-   ##
=======================================
  Coverage   52.17%   52.17%           
=======================================
  Files          10       10           
  Lines         621      621           
  Branches      156      156           
=======================================
  Hits          324      324           
  Misses        297      297
Impacted Files Coverage Δ
src/main.ts 88.63% <0%> (ø) ⬆️

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 f9f7cfc...87fceb4. Read the comment docs.

@matt-gadd matt-gadd merged commit 507e2fb into dojo:master Mar 7, 2019
agubler pushed a commit to agubler/cli-build-app that referenced this pull request Mar 8, 2019
* change to using req.accepts

* package-lock
agubler added a commit that referenced this pull request Mar 21, 2019
* Add slash to paths that need them in serve (#247)

* add slash to paths that need them

* only rewrite on html type requests

* fix test

* Change path rewriting to use req.accepts in serve mode (#253)

* change to using req.accepts

* package-lock
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.

2 participants