Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SetValue for TwoOptions doesn't work for checkboxes #404

Closed
mVermaat opened this issue Mar 5, 2019 · 5 comments
Closed

SetValue for TwoOptions doesn't work for checkboxes #404

mVermaat opened this issue Mar 5, 2019 · 5 comments
Assignees
@jlattimer
Labels
bug

Comments

@mVermaat
Copy link
Contributor

mVermaat commented Mar 5, 2019

When using the "SetValue(TwoOption option)" method, it doesn't properly work for checkboxes. It clicks next to the box, so the box doesn't get (un)checked.

It currently clicks the fieldelement and not the imput element that is a child of the fieldelement. the fieldelement is too big, so it clicks next to the box. If you click on the input element, it seems to work.
@NicolasPrats
Copy link

Hello,

We have the same issue. However the deprecated method SetValue(string, bool) is working fine.

@dtu11
Copy link
Member

dtu11 commented Mar 6, 2019

Thanks Nicolas for a workaround.

Which version and which client are you working with?

@dtu11 dtu11 added the bug label Mar 6, 2019
@NicolasPrats
Copy link

Hello @dtu11 ,

We are working with Chrome and D365 Version 1710 (9.1.0.2640) online.

@mVermaat
Copy link
Contributor Author

mVermaat commented Mar 7, 2019

@dtu11

For me exactly the same. Good plan to use the deprecated method for now.

@mVermaat mVermaat mentioned this issue Mar 12, 2019
Closed
@jlattimer
Copy link
Contributor

When formatted as a list it doesn't work either. Going from false to true opens the drop down but never sets. Going from true to false doesn't open the drop down at all.

@jlattimer jlattimer self-assigned this May 17, 2019
@jlattimer jlattimer mentioned this issue May 19, 2019
Merged
TYLEROL pushed a commit that referenced this issue May 21, 2019
@jlattimer @TYLEROL
GitHub issues #404 and #380 Completing boolean Gets & Sets for all 3 …
8808642 
…field formats. Sorry this is a lot at once but I thought it made sense to keep it together (#483)
TYLEROL pushed a commit that referenced this issue May 21, 2019
@jlattimer @TYLEROL
GitHub issues #404 and #380 Completing boolean Gets & Sets for all 3 …
78fca2f 
…field formats. Sorry this is a lot at once but I thought it made sense to keep it together (#483)
TYLEROL added a commit that referenced this issue May 21, 2019
@TYLEROL
Dev/tylerol/cherry pick483to v91 (#491)
44558b6 
* GitHub issues #404 and #380 Completing boolean Gets & Sets for all 3 field formats. Sorry this is a lot at once but I thought it made sense to keep it together (#483)
TYLEROL added a commit that referenced this issue May 22, 2019
@TYLEROL
Merging changes from develop into v9.1 (#500)
180d310 
* Merging alpha branch with new develop branch (#457)

* Adding support for Azure DevOps build testing

* Merging UCI Nav Changes (#453)

* Changes for Forcing UCI from Main.aspx

* Updates

* Committing

* Changes for Missing Switcher in SiteMap

* Remove project

* Delete Documentation.csproj

* Delete Program.cs

* Delete App.config

* Delete AssemblyInfo.cs

* Delete app.config

* Updated App.Config

* App.Config

* Revert "Merging UCI Nav Changes (#453)" (#454)

This reverts commit 88d60cd.

* Perf Enhancement to OpenSubArea (#458)

* Merging alpha branch with new develop branch (#457)

* Adding support for Azure DevOps build testing

* Merging UCI Nav Changes (#453)

* Changes for Forcing UCI from Main.aspx

* Updates

* Committing

* Changes for Missing Switcher in SiteMap

* Remove project

* Delete Documentation.csproj

* Delete Program.cs

* Delete App.config

* Delete AssemblyInfo.cs

* Delete app.config

* Updated App.Config

* App.Config

* Revert "Merging UCI Nav Changes (#453)" (#454)

This reverts commit 88d60cd.

* Adding WaitUntilAvailable to ensure the Timeline popout menu is both opened and clicked, allowing SetValue to work as expected. (#465)

* Update Grid.Search() method to match org sample data (#467)

* GitHub issues #404 and #380 Completing boolean Gets & Sets for all 3 field formats. Sorry this is a lot at once but I thought it made sense to keep it together (#483)

* GitHub issue #439 Fixed OpenEntity to open specified record instead of re-opening the current record (#484)

* GitHub issue #452 Added UCI  new method to use form selector to change forms (#486)

* Added support for remote execution (#475) (#493)

Added remote execution support for selenium grid for Firefox and Chrome web browsers. This is useful when you need to perform a large number of tests.

* Allows browser height & width to be set manualy to a larger size than the current resolution supports. (#476) (#494)

* Updated SetValue for lookup to use the parameter value to match in the dialog versus only looking at the immediate results. (#479) (#495)

Updated SelectLookup function to utilize previously unused parameter to prevent lookup dialog from opening after clearing the field.

* GitHub issues #394 & #7 Fixes issues with UCI fields not clearing or not removing previous value when updating (#481) (#496)

* GitHub issue #371 Updated SetValue for lookup. Added a WaitForTransaction to ensure the page is loaded to prevent issue with SetValue executing when the load is still happening. Added sleeps to make sure the elements get hovered & clicked properly (#480) (#497)

* Merge into v91
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jlattimer jlattimer

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@jlattimer @mVermaat @dtu11 @NicolasPrats