Skip to content

more update(el, opts) changes#1491

Merged
adumesny merged 1 commit into
gridstack:developfrom
adumesny:develop
Nov 29, 2020
Merged

more update(el, opts) changes#1491
adumesny merged 1 commit into
gridstack:developfrom
adumesny:develop

Conversation

@adumesny

@adumesny adumesny commented Nov 29, 2020

Copy link
Copy Markdown
Member

Description

more for fix #1235

  • moved the old individual routines to be hidden as update() does everything now
  • updated doc which I forgot on last check in
  • removed some old 0.5/0.6 backward compatible checks

Checklist

  • Created tests which fail without the change (if possible)
  • All tests passing (yarn test)
  • Extended the README / documentation, if necessary

more for fix gridstack#1235
* moved the old individual routines to be hidden as update() does everything now
* updated doc which I forgot on last check in
* removed some old 0.5/0.6 backward compatible checks
@adumesny adumesny merged commit fa8634c into gridstack:develop Nov 29, 2020
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.

update() should also take the options object

1 participant