You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Got feedback from @dvoytenko on this (quote: a POST without user action is generally suspicious).
Currently I switched the request to a POST so that we're able to transmit the json data { ampUserId, elementId } to the server and it can determine wether to show the notification to the current visitor or not.
The alternative is to switch back to a GET request and transmit the data as query params.
Got feedback from @dvoytenko on this (quote: a POST without user action is generally suspicious).
Currently I switched the request to a POST so that we're able to transmit the json data
{ ampUserId, elementId }
to the server and it can determine wether to show the notification to the current visitor or not.The alternative is to switch back to a
GET
request and transmit the data as query params./cc @dvoytenko @cramforce for input
The text was updated successfully, but these errors were encountered: