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 link to Report a Bug form #77

Merged
merged 1 commit into from Sep 7, 2014
Merged

Conversation

Niharika29
Copy link
Contributor

Some information is collected and sent as URL parameters as well. The current link points to bugtracker.timvideos.us instance which needs to be redirected to the current instance of the app.

How the link looks (placed in the footer):
footer_tv

What the URL looks like after clicking on the link:
url_tv

@@ -41,7 +41,6 @@

from .data import data


Copy link
Member

Choose a reason for hiding this comment

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

Can you please remove this change from this commit?

@mithro
Copy link
Member

mithro commented Aug 17, 2014

Please squash all your commits into a single commit.

Please also add a screenshot to your pull request so we can see what this looks like.

Thanks!

@mithro
Copy link
Member

mithro commented Aug 21, 2014

Hi @Niharika29,

Could you please fix the commit message to be a bit more informative rather then just all your commit messages in one. Once that is done then this patch looks is good to merge!

What do we need to do to get bugtracker.timvideos.us URL working?

Tim

Working state

Sending info from streaming-system using GET

Wrap-up sending variables using GET

Fetch stream title/url

Deleted temporary files

More temporary files

Making patch ready for merge

Typo

Styling

Cleaning up

Change url to timvideos.us for nginx redirection

Fix stuff
mithro added a commit that referenced this pull request Sep 7, 2014
Add link to Report a Bug form
@mithro mithro merged commit 3981906 into timvideos:master Sep 7, 2014
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.

None yet

2 participants