Skip to content

Fix Static files route not working #1671

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

Merged
merged 4 commits into from
Dec 16, 2020
Merged

Fix Static files route not working #1671

merged 4 commits into from
Dec 16, 2020

Conversation

pwli0755
Copy link
Contributor

@pwli0755 pwli0755 commented Nov 6, 2020

fix 404 error when using Static files route Echo.Static()

related issue

@codecov
Copy link

codecov bot commented Nov 6, 2020

Codecov Report

Merging #1671 (2374af4) into master (4422e3b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1671   +/-   ##
=======================================
  Coverage   85.19%   85.20%           
=======================================
  Files          29       29           
  Lines        1986     1987    +1     
=======================================
+ Hits         1692     1693    +1     
  Misses        186      186           
  Partials      108      108           
Impacted Files Coverage Δ
echo.go 86.44% <100.00%> (+0.04%) ⬆️

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 4422e3b...2374af4. Read the comment docs.

Copy link
Contributor

@pafuent pafuent left a comment

Choose a reason for hiding this comment

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

Thanks for upgrading your PR with the changes on master!!!

@pwli0755
Copy link
Contributor Author

@pafuent

can you merge? to avoid incoming conflict

@pafuent
Copy link
Contributor

pafuent commented Dec 16, 2020

Thanks for your contribution!!!

@pafuent pafuent merged commit 0482cb3 into labstack:master Dec 16, 2020
@pwli0755 pwli0755 deleted the fix-conflict branch December 16, 2020 04:31
aldas added a commit to aldas/echo that referenced this pull request Dec 17, 2020
@lammel
Copy link
Contributor

lammel commented Dec 17, 2020

Due to an open issue with a trailing slash the PR needs to be resubmitted @pwli0755 .
Could you please create a new PR based on your branch again.

Sorry for the inconvenience.

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.

3 participants