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

feat(api:) arg order #268

Merged
merged 2 commits into from
Oct 7, 2016
Merged

feat(api:) arg order #268

merged 2 commits into from
Oct 7, 2016

Conversation

yoshuawuyts
Copy link
Member

Changes the argument order. Closes #179

Changes

  • swap argument order ✨
  • replace last remaining action arguments with data for consistency
  • update to barracks@9
  • fixed a stubbed test (which was hanging)
  • ref: [release] 4.0 tracker #204

@yoshuawuyts
Copy link
Member Author

Merging as there's very few interesting things going on here; mostly a regex replace - not going to release v4 yet tho soooo can merrily merge and wait for humans to review-after-merge hah

@yoshuawuyts yoshuawuyts merged commit 1700ac5 into 4 Oct 7, 2016
@yoshuawuyts yoshuawuyts deleted the arg-order branch October 7, 2016 22:51
@yoshuawuyts yoshuawuyts added this to the 4.0.0 milestone Oct 7, 2016
yoshuawuyts added a commit that referenced this pull request Oct 16, 2016
* s/data, state/state, data/

* feat(api): swap arguments
yoshuawuyts added a commit that referenced this pull request Dec 11, 2016
* s/data, state/state, data/

* feat(api): swap arguments
yoshuawuyts added a commit that referenced this pull request Dec 11, 2016
* location: change url on location:setLocation

- [ ] don't break hashing
- [ ] allow not changing the url

fixup! move fns around

fixup! add search string

* app.start: clean

* walk: add

* location: update arg calls

* fixup! location: update

* uri-wrap: fix thunking

* router: update to latest sheet-router (#239)

* router: update to latest sheet-router

* tests: fix for latest version

* tests: fix SSR

* tests: fix history

* tests: spruce up

* docs: update example

* examples: update

* deps: bump sheet-router

* 4.0.0-0

* chore(changelog): 4.0.0 (#211)

* feat(api:) arg order (#268)

* s/data, state/state, data/

* feat(api): swap arguments

* fix(href): fix routing (#271)

* feat(http): remove (#269)

* feat(router): enable hash routing (#273)

* deps: fix mount

* 4.0.0-1

* feat(mount): copy {script,link} tags

* 4.0.0-2

* fix(mount): forEach -> for

Lol can't use forEach

* fix(router): use state.location.href (#282)

* fix(mount): use deep node clone

* 4.0.0-3

* fix(deps): remove hash-match

* 4.0.0-4

* fix(mount): return node

* 4.0.0-5

* fix(router): check if a hash is a valid selector (#339)

* 4.0.0-6

* fix(router): pass params on newstate (#343)

* 4.0.0-7

* feat(docs): update for 4.0.0 (#320)

* feat(docs): update for 4.0.0

* docs: update router example in readme (#337)

* chore(changelog): update for v4 (#351)
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