{"action":"created","issue":{"url":"https://api.github.com/repos/plone/mockup/issues/370","labels_url":"https://api.github.com/repos/plone/mockup/issues/370/labels{/name}","comments_url":"https://api.github.com/repos/plone/mockup/issues/370/comments","events_url":"https://api.github.com/repos/plone/mockup/issues/370/events","html_url":"https://github.com/plone/mockup/pull/370","id":30585278,"number":370,"title":"feat(modal): Allow not to use ajax","user":{"login":"do3cc","id":147512,"avatar_url":"https://avatars.githubusercontent.com/u/147512?v=3","gravatar_id":"","url":"https://api.github.com/users/do3cc","html_url":"https://github.com/do3cc","followers_url":"https://api.github.com/users/do3cc/followers","following_url":"https://api.github.com/users/do3cc/following{/other_user}","gists_url":"https://api.github.com/users/do3cc/gists{/gist_id}","starred_url":"https://api.github.com/users/do3cc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/do3cc/subscriptions","organizations_url":"https://api.github.com/users/do3cc/orgs","repos_url":"https://api.github.com/users/do3cc/repos","events_url":"https://api.github.com/users/do3cc/events{/privacy}","received_events_url":"https://api.github.com/users/do3cc/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2014-04-01T09:56:29Z","updated_at":"2015-04-26T19:01:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/plone/mockup/pulls/370","html_url":"https://github.com/plone/mockup/pull/370","diff_url":"https://github.com/plone/mockup/pull/370.diff","patch_url":"https://github.com/plone/mockup/pull/370.patch"},"body":"This patch offers two things.\r\n1. Allow not to use ajax for forms. Useful for the delete confirmation\r\n   popup\r\n2. If a page does not have a base tag, find the content url from\r\n   $(body).data('base-url')\r\n   The data attribute has been added in plone.app.layout 2.5.1.dev0\r\n\r\nBackground on the delete confirmation:\r\nWe wanted to just redirect on success, but this looses features with ajax.\r\nThere is no way to get the statusmessage. The statusmessage is set on the\r\n302 reply, then jquery directly follows the redirect and receives the\r\ninformation that it shall delete the cookie. Now the success handler has\r\nno trace of the status message. After thinking of patching statusmessage\r\nto maybe add the status message just once as a response header field, I\r\nrealized I could just not use ajax on form submit since it offers no\r\nadvantage for this use case."},"comment":{"url":"https://api.github.com/repos/plone/mockup/issues/comments/96420967","html_url":"https://github.com/plone/mockup/pull/370#issuecomment-96420967","issue_url":"https://api.github.com/repos/plone/mockup/issues/370","id":96420967,"user":{"login":"jensens","id":157140,"avatar_url":"https://avatars.githubusercontent.com/u/157140?v=3","gravatar_id":"","url":"https://api.github.com/users/jensens","html_url":"https://github.com/jensens","followers_url":"https://api.github.com/users/jensens/followers","following_url":"https://api.github.com/users/jensens/following{/other_user}","gists_url":"https://api.github.com/users/jensens/gists{/gist_id}","starred_url":"https://api.github.com/users/jensens/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jensens/subscriptions","organizations_url":"https://api.github.com/users/jensens/orgs","repos_url":"https://api.github.com/users/jensens/repos","events_url":"https://api.github.com/users/jensens/events{/privacy}","received_events_url":"https://api.github.com/users/jensens/received_events","type":"User","site_admin":false},"created_at":"2015-04-26T19:01:52Z","updated_at":"2015-04-26T19:01:52Z","body":"whats the state now here?"}}