Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 00:14
    greenkeeper[bot] labeled #5449
  • 00:14
    greenkeeper[bot] opened #5449
  • 00:14

    greenkeeper[bot] on can-super-model-2.0.0

    fix(package): update can-super-… (compare)

  • 00:13
    greenkeeper[bot] labeled #5448
  • 00:13
    greenkeeper[bot] opened #5448
  • 00:13

    greenkeeper[bot] on can-define-rest-model-2.0.0

    fix(package): update can-define… (compare)

  • 00:12
    greenkeeper[bot] labeled #5447
  • 00:12
    greenkeeper[bot] opened #5447
  • 00:12

    greenkeeper[bot] on can-define-realtime-rest-model-2.0.0

    fix(package): update can-define… (compare)

  • 00:07
    greenkeeper[bot] labeled #5446
  • 00:07
    greenkeeper[bot] opened #5446
  • 00:07

    greenkeeper[bot] on can-connect-ndjson-2.0.0

    fix(package): update can-connec… (compare)

  • Dec 13 21:27

    greenkeeper[bot] on @octokit

    (compare)

  • Dec 13 21:25

    greenkeeper[bot] on @octokit

    chore(package): update @octokit… (compare)

  • Dec 13 21:21

    greenkeeper[bot] on @octokit

    (compare)

  • Dec 13 21:20

    greenkeeper[bot] on @octokit

    chore(package): update @octokit… (compare)

  • Dec 12 19:26
    cherifGsoul review_requested #5443
  • Dec 12 18:34
    cherifGsoul synchronize #5443
  • Dec 12 18:34

    cherifGsoul on document-inserted-removed-can6

    Update after review (compare)

  • Dec 12 17:44

    cherifGsoul on theme-3.0.0-pre.5

    (compare)

Kevin Phillips
@phillipskevin
nevertheless, you get events based off the diff
Thomas Sieverding
@Bajix
I was just thinking… with your earlier example wherein you used an object to do the filter - that would be way more efficient if you were returning a new List with a merged set
Kevin Phillips
@phillipskevin
I’d have to try it out
Thomas Sieverding
@Bajix
get active() { return new List(Object.assign({}, this.__listSet, { active: true })); }
Kevin Phillips
@phillipskevin
I don’t know that I understand listSet well enough… what would that do exactly?
Nico R.
@nriesco

@phillipskevin I saw your example and those are great examples. On knockoutjs you could define the way this kind of “computed” attributes were updated, can you do that in can?

this kind of thing is what I use most often

  get active(){
    return this.filter({complete: false});
  },
  get complete(){
    return this.filter({complete: true});
  },
  get allComplete(){
    return this.length === this.complete.length;
  }
Thomas Sieverding
@Bajix
It’s the set Algebra used to query against
Nico R.
@nriesco
you could for example wait 100 milliseconds before the update or stuff like that
Thomas Sieverding
@Bajix
So you’d be returning a new list that’s a subset of the original list, and you’d just rely that they’d be managed via can-connect
can-stream does that
Nico R.
@nriesco
I’m worried because when dealing with lists of complex objects it would get slow when dealing with small sets of say 30 or 50 causing the application to use a lot of cpu
Kevin Phillips
@phillipskevin
you could do that manually
using an asynchronous getter
but I would wait until you run into an issue like that before doing that kind of thing
most of the time, you should let the event batching system take care of that kind of thing for you
@Bajix wouldn’t this.__listSet tell can-connect that it’s the same list?
Thomas Sieverding
@Bajix
@phillipskevin You’d merge the two sets
Nico R.
@nriesco
@phillipskevin ok thanks!
Kevin Phillips
@phillipskevin
ok @Bajix I’m sure I’ll understand some day
for now I’m going to take your word for it :smile:
Thomas Sieverding
@Bajix
Well that works too
That’s how it works ^
Or at least that would clarify
Kevin Phillips
@phillipskevin
thanks
Thomas Sieverding
@Bajix
@phillipskevin I don’t suppose you could do a release w/ canjs/steal-stache#11 & canjs/can-connect#210 merged?
Kevin Phillips
@phillipskevin
individual releases of those? or a canjs/canjs release?
Thomas Sieverding
@Bajix
Just those
They are forcing an older version of jQuery
Kevin Phillips
@phillipskevin
yeah
sorry, yeah I know that’s the issue
let me see if I can get added to steal-stache
Thomas Sieverding
@Bajix
Cool thanks
Kevin Phillips
@phillipskevin
any reason to use ^3.1.1 || ^2.2.4 instead of 2.x - 3.x
Thomas Sieverding
@Bajix
When I had tested it on steal-stache, 2.x - 3.x was installing 2.2.4, but perhaps that was from the circular reference with can-view-imports
There doesn’t seem to be any good reason to support older 3.x or 2.x
I was hoping that this syntax would be better for Travis CI
Kevin Phillips
@phillipskevin
just that people might already have them as dependencies
Thomas Sieverding
@Bajix
That wouldn’t matter
Kevin Phillips
@phillipskevin
I don’t think I’ll be able to release them today
Thomas Sieverding
@Bajix
Is that just because of the above?
What I mean to say is that it’s a non-issue with people already having them as dependencies as the existing projects would be semantically compatible anyway
Kevin Phillips
@phillipskevin
yeah, I understand what you mean
just not sure if @justinbmeyer cares one way or the other
so I’d rather have him dismiss his review
also, it’s time for dinner
Thomas Sieverding
@Bajix
I can just do 2.x - 3.x if that means we can release ASAP
Kevin Phillips
@phillipskevin
ok, sure
Thomas Sieverding
@Bajix
Done
Kevin Phillips
@phillipskevin