Skip to content
This repository was archived by the owner on Jan 31, 2018. It is now read-only.

1003288 capture user agent#291

Closed
willkg wants to merge 2 commits intomozilla:masterfrom
willkg:1003288-capture-user-agent
Closed

1003288 capture user agent#291
willkg wants to merge 2 commits intomozilla:masterfrom
willkg:1003288-capture-user-agent

Conversation

@willkg
Copy link
Copy Markdown
Member

@willkg willkg commented May 22, 2014

Fix the FxOS form so that it captures the user-agent. This also required adding a user-agent field to the API.

Quick r?

@willkg
Copy link
Copy Markdown
Member Author

willkg commented May 22, 2014

To test:

  1. run tests
  2. run the server, go to http://localhost:8000/feedback/fxos and fill out the form
  3. log in and then look at the response and verify it has a user agent

@rlr
Copy link
Copy Markdown
Contributor

rlr commented May 22, 2014

WFM R+!

@willkg
Copy link
Copy Markdown
Member Author

willkg commented May 22, 2014

2eda162 [bug 1003288] Add user-agent to FxOS form
2167bae [bug 1003288] Add user_agent field to API

@willkg willkg closed this May 22, 2014
@willkg willkg deleted the 1003288-capture-user-agent branch May 22, 2014 16:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants