-
Notifications
You must be signed in to change notification settings - Fork 13
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.
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
Add support for conditional clicking #3623
base: main
Are you sure you want to change the base?
Conversation
I need something like this here, but keep getting stuck: // Explicitly encode `null` if `defaultClick` is nil
if let defaultClick = defaultClick {
try container.encode(defaultClick, forKey: .defaultClick)
} else {
try container.encodeNil(forKey: .defaultClick)
} |
This PR has been inactive for more than 7 days and will be automatically closed 7 days from now. |
This PR has been closed after 14 days of inactivity. Feel free to reopen it if you plan to continue working on it or have further discussions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@brianhall I've got as far as step #9 when testing, but I'm seeing a URL with #1 at the end. I thought it might be some caching issue, so I tried renaming the test broker, but it still happened
@THISISDINOSAUR Did you happen to update the JSON in step 8? I should have mentioned it, but you then have to re-build the browser to get the latest JSON. Will add that in the instructions now. |
@brianhall Yeah I did, I tried a clean build too and renaming the that broker just in case it was being cached. |
@THISISDINOSAUR Yep, that’s exactly right. I made a few other updates to the instructions to hopefully make them a bit clearer. Just to clarify, you should see a |
Task/Issue URL: https://app.asana.com/0/1199230911884351/1208647902506360/f
Tech Design URL:
CC:
Description:
Adds support for conditional clicking of elements by adding “choices” and “default” keys to the click action.
Optional E2E tests:
Check this to run the Personal Information Removal end to end tests. If updating CCF, or any PIR related code, tick this.
Steps to test this PR:
The easiest way to test whether the decoding is working is to setup one of the test files from the C-S-S integration tests as a fake broker:
file://
LocalPackages/DataBrokerProtection/Sources/DataBrokerProtection/Resources/JSON
main
), then open the debug version of PIR: Debug Menu -> Personal Information Removal -> Run Personal Information Removal in Debug Mode.1965
, then select Test Broker in the list of brokers and click the Run button.window.location.href
and hit enter. You should see a url that ends with#1
, which means that the user was older than 45.1990
, when you dowindow.location.href
you should get a url that ends with#2
instead, indicating that the user is < 45.null
. Now re-build the browser to load the updated JSON.window.location.href
should return a url with#1
for step 5, and a url with no#
at the end for step 7, meaning that nothing was clicked and no error was triggered.#1
hash at the end of the url), and step 7 should not return an error, but also not have any hash (e.g.#1
) at the end of the url.Definition of Done:
—
Internal references:
Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation