[GitHub] [cloudstack] shwstppr opened a new issue #6184: ui: latest ui sends undesired api parameters on form submit

[GitBox <gi...@apache.org>]

shwstppr opened a new issue #6184:
URL: https://github.com/apache/cloudstack/issues/6184


   <!--
   Verify first that your issue/request is not already reported on GitHub.
   Also test if the latest release and main branch are affected too.
   Always add information AFTER of these HTML comments, but no need to delete the comments.
   -->
   
   ##### ISSUE TYPE
   <!-- Pick one below and delete the rest -->
    * Bug Report
   
   ##### COMPONENT NAME
   <!--
   Categorize the issue, e.g. API, VR, VPN, UI, etc.
   -->
   ~~~
   UI
   ~~~
   
   ##### CLOUDSTACK VERSION
   <!--
   New line separated list of affected versions, commit ID for issues on main branch.
   -->
   
   ~~~
   main
   ~~~
   
   ##### CONFIGURATION
   <!--
   Information about the configuration if relevant, e.g. basic network, advanced networking, etc.  N/A otherwise
   -->
   N/A
   
   ##### OS / ENVIRONMENT
   <!--
   Information about the environment if relevant, N/A otherwise
   -->
   N/A
   
   ##### SUMMARY
   <!-- Explain the problem/feature briefly -->
   With vue3 changes, on submitting form UI sends undesired API parameters. UI element for these parameters are not shown in the form due to appropriate conditions but on form submit they are still sent if they appear once. Such elements are rendered conditionally with `v-if` tag
   
   ##### STEPS TO REPRODUCE
   <!--
   For bugs, show exactly how to reproduce the problem, using a minimal test-case. Use Screenshots if accurate.
   
   For new features, show how the feature would be used.
   -->
   
   <!-- Paste example playbooks or commands between quotes below -->
   ~~~
   - Go to a UI form which shows some elements conditionally
   - Initially select options that enable those conditions
   - Then select options which doesn't enable those conditions
   - Submit the form
   - Observe in API call made by the UI that parameters that should be sent are present there
   ~~~
   
   https://user-images.githubusercontent.com/153340/161016269-5322f93b-4f0d-4d5d-a0ad-82a795ba7f56.mp4
   
   In the example `deployasis` parameter is passed though it should not be.
   
   <!-- You can also paste gist.github.com links for larger files -->
   
   ##### EXPECTED RESULTS
   <!-- What did you expect to happen when running the steps above? -->
   
   ~~~
   API call should be made only with desired parameters
   ~~~
   
   ##### ACTUAL RESULTS
   <!-- What actually happened? -->
   
   <!-- Paste verbatim command output between quotes below -->
   ~~~
   Undesired parameters are present in the API call. This could potentially break some functionalities
   ~~~
   


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

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