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

Display agent success / error messages in UI #735

Merged
merged 5 commits into from Apr 7, 2015
Merged

Conversation

@ashic
Copy link
Contributor

ashic commented Apr 3, 2015

  • This feature wasn't working in the current repo. Seems like a jQuery fileupload issue. This pull request fixes it.

Also fixes issue where agent with no deployments showed errors on the agent page.

ashic added 5 commits Apr 3, 2015
…ed is called fields
Currently, no arguments are passed to the agent from
Deploy.Web. This pull request passes env=EnvironmentName
for the agent. This lets us parameterize deployments.

Currently, the mechanism of passing multiple arguments
alongside a package POST adds them as a ; separated string,
which is then available in the deployment script verbatim.
This pull request changes to the approach outlined by RFC 2616
(http://www.w3.org/Protocols/rfc2616/rfc2616-sec4.html) where
multiple values are added to the header comma separated and
quoted, with only quote characters percent encoded. To support
existing clients, POSTs without comma separated quoted format
are still passed verbatim.
@forki forki merged commit 8377133 into fsharp:master Apr 7, 2015
2 checks passed
2 checks passed
continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.