HomePhabricator

Transactions - fix edit submit URI

Description

Transactions - fix edit submit URI

Summary:
tracing this a bit, in core/behavior-form.js we end up doing form.submit() on this form. Before this patch, the form had no submitURI set so it ended up doing a page load. Now, we set the submitURI and (at least) the form is actually submitted and data is saved. Ref T4669.

Not sure how come the syntheticSubmit even isn't getting picked up though?

Test Plan: hit apple + enter to submit comment edit. page reloaded and my edit persisted!

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: epriestley, Korvin

Maniphest Tasks: T4669

Differential Revision: https://secure.phabricator.com/D8952

Details

Provenance
btrahanAuthored on
btrahanPushed on May 2 2014, 4:37 PM
Reviewer
epriestley
Differential Revision
D8952: Transactions - fix edit submit URI
Parents
rP97f88f468b31: Audit - fix profile link
Branches
Unknown
Tags
Unknown
Tasks
T4669: Submitting comment edits with ctrl+enter doesn't ajax

Event Timeline