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.

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 use_resetter and wait_for_view kwargs #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mshriver
Copy link
Contributor

control navigation via kwargs on navigate()
use_resetter: Whether or not resetter function is called
wait_for_view: timeout value for am_i_here to be true after post_navigate

use_resetter defaults to True, current behavior maintained
wait_for_view defaults to None, current behavior maintained

FIXES #25

@mshriver
Copy link
Contributor Author

@psav Thoughts?

@mshriver mshriver force-pushed the add-navigate-kwargs branch from d07ca46 to 68ca832 Compare January 16, 2019 16:00
tests/test_simple.py Outdated Show resolved Hide resolved
@mshriver mshriver force-pushed the add-navigate-kwargs branch 2 times, most recently from dccd05b to d40c69e Compare January 16, 2019 16:09
@oshtaier
Copy link

@mshriver @psav ping

@mshriver mshriver force-pushed the add-navigate-kwargs branch from 279fbcf to 8c92ea3 Compare December 13, 2023 13:41
@mshriver mshriver marked this pull request as draft December 13, 2023 13:41
@mshriver mshriver force-pushed the add-navigate-kwargs branch 3 times, most recently from 7ba15e1 to e45ad9d Compare December 13, 2023 16:02
@mshriver mshriver marked this pull request as ready for review December 13, 2023 16:04
@mshriver mshriver force-pushed the add-navigate-kwargs branch from 0d389bc to adca211 Compare December 13, 2023 16:05
Copy link
Collaborator

@RonnyPfannschmidt RonnyPfannschmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Next week I'll have the mental capacity to compare this with some hacks in iqe as im under the impression some passover is missing

I'd love to dig in Sooner but I do have to recover from the flu better first

src/navmazing/_navigator.py Outdated Show resolved Hide resolved
@mshriver mshriver marked this pull request as draft December 15, 2023 12:33
@mshriver mshriver force-pushed the add-navigate-kwargs branch from adca211 to becbacd Compare December 15, 2023 12:35
control navigation via kwargs on navigate()
use_resetter:  Whether or not resetter function is called
wait_for_view: timeout value for am_i_here to be true after post_navigate

use_resetter defaults to True, current behavior maintained
wait_for_view defaults to None, current behavior maintained

Add NAVIGATE logging messages for both
@mshriver mshriver force-pushed the add-navigate-kwargs branch from af2484f to c020287 Compare December 15, 2023 12:40
@mshriver mshriver marked this pull request as ready for review December 15, 2023 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support control of resetter and am_i_here wait in go()
3 participants