lukaszlenart commented on a change in pull request #154:
URL: https://github.com/apache/struts-site/pull/154#discussion_r541910942



##########
File path: source/tag-developers/set-tag.md
##########
@@ -25,9 +25,10 @@ The scopes available are as follows:
 - `session` - the value will be set in session scope according to servlet 
spec. using the name as key
 - `request` - the value will be set in request scope according to servlet 
spec. using the name as key
 - `page` - the value will be set in page scope according to servlet spec. 
using the name as key
-- `action` - the value will be set in the request scope and Struts' action 
context using the name as key
+- `action` - the value will be set in the page scope and Struts' action 
context using the name as key

Review comment:
       Not really, as you can chain actions so this will change which one is on 
the top of ValueStack.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org

Reply via email to