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

Pass state between requests #165

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

peldax
Copy link

@peldax peldax commented Jan 22, 2020

  • Remebers states recieved from nette (merge array instead of replace).
  • Updates URL to add all persistent paramers.
  • Can be disabled using settings noState or data attribute data-no-state.

Some work is still needed to detect when nette drops parameter (-> uses default one).

Its big BC break. I am just leaving it here for anyone interested.

@peldax
Copy link
Author

peldax commented Jan 22, 2020

Switched back to replace current state (original commit performed merge).

This way we dont need to detect Nette dropping default parameters, because when we pass all peristent parameters in request Nette returns everything back in its up to date state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant