-
Notifications
You must be signed in to change notification settings - Fork 21.6k
swarm: Add favicon for Swarm templates served by browser #15958
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
Conversation
karalabe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SGTM
|
@karalabe Thanks so much! I so appreciate it. I also have another PR waiting for review also related to http interface of Swarm, ethereum#15926...would you mind taking a look at that? Thanks again! |
karalabe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, the swarm tests fail on this one. Please fix those.
|
@karalabe will do! on it now |
|
Ping? |
|
hey @karalabe ! Thanks for following up. Swarm Dev incorrectly stated it was an error of whisper protocol (which is true for one test shown below) but another test fails in server_test.go, which I just realized while typing this to you. Working on it now!
|
|
Tests are still failing. |
|
working on it now. apologies for the delay but i'm a community contributor,
not part of ethereum foundation
…On Mon, Feb 5, 2018 at 6:29 PM, Péter Szilágyi ***@***.***> wrote:
Tests are still failing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#15958 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHxGKQYyFaDU3yXHYpPwvTFPWxBv7gu0ks5tR5zVgaJpZM4RqLhc>
.
|
|
Thank you for your contribution! Your commits seem to not adhere to the repository coding standards
Please check the contribution guidelines for more details. This message was auto-generated by https://gitcop.com |
|
@karalabe main tests are fixed, rebasing from master right now to correct one more. |
|
@karalabe all the swarm tests are passing -- the fail test is coming from |
karalabe
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
@karalabe thank you!! |
) * swarm: added script to HTML header to create favicon addresses ethereum#153 * swarm: moved data blob direclty into link tag, removed script * swarm: added favicon info to other html templates * swarm: fixing test errors * swarm: fixing favicon test * swarm: fixing travis tests * swarm: added script to HTML header to create favicon addresses ethereum#153 * swarm: moved data blob direclty into link tag, removed script * swarm: added favicon info to other html templates * swarm: fixing test errors * swarm: fixing favicon test * swarm: fixing travis tests
) * swarm: added script to HTML header to create favicon addresses ethereum#153 * swarm: moved data blob direclty into link tag, removed script * swarm: added favicon info to other html templates * swarm: fixing test errors * swarm: fixing favicon test * swarm: fixing travis tests * swarm: added script to HTML header to create favicon addresses ethereum#153 * swarm: moved data blob direclty into link tag, removed script * swarm: added favicon info to other html templates * swarm: fixing test errors * swarm: fixing favicon test * swarm: fixing travis tests
This PR adds a base64 data blob in the
headof the generic HTML templates used by the HTTP Swarm client.This PR has been approved by 2 members of the Swarm development team in ethersphere#208.
It is a contribution to the goal of improving the http interface for Swarm users. Please let me know if you have any questions.