Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 18 22:20
    greenkeeper[bot] labeled #5391
  • Oct 18 22:20
    greenkeeper[bot] opened #5391
  • Oct 18 22:20

    greenkeeper[bot] on can-observable-mixin-1.0.3

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

  • Oct 18 19:08
    cherifGsoul labeled #5390
  • Oct 18 19:07
    cherifGsoul opened #5390
  • Oct 18 18:45
    greenkeeper[bot] labeled #5389
  • Oct 18 18:45
    greenkeeper[bot] opened #5389
  • Oct 18 18:45

    greenkeeper[bot] on can-observable-bindings-1.3.0

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

  • Oct 18 18:29

    matthewp on master

    Update dist for release (compare)

  • Oct 18 18:29

    matthewp on v6.1.3

    Update dist for release 6.1.3 (compare)

  • Oct 18 17:59

    matthewp on can-observable-array-1.0.2

    (compare)

  • Oct 18 17:59

    matthewp on master

    fix(package): update can-observ… Merge pull request #5388 from c… (compare)

  • Oct 18 17:59
    matthewp closed #5388
  • Oct 18 16:46
    greenkeeper[bot] labeled #5388
  • Oct 18 16:46
    greenkeeper[bot] opened #5388
  • Oct 18 16:46

    greenkeeper[bot] on can-observable-array-1.0.2

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

  • Oct 18 12:45
    greenkeeper[bot] labeled #5387
  • Oct 18 12:45
    greenkeeper[bot] opened #5387
  • Oct 18 12:45

    greenkeeper[bot] on can-stache-element-1.0.2

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

  • Oct 18 09:49
    chasenlehara edited #5384
nmingneau
@nmingneau
Does anyone know if there is a way to disable the logging in fixtures when they receive a call?
Justin Meyer
@justinbmeyer
There is not. Easy to add. Why do you want it?
nmingneau
@nmingneau
yes please, it takes a lot of space in my unit tests
Justin Meyer
@justinbmeyer
@nmingneau can you create an issue to enable a fixture.log(false) ?

generally speaking, it will be as simple to add as finding these logs:

https://github.com/canjs/can-fixture/blob/master/core.js#L197

and wrapping them with a check if(fixture._log) { ... }
RyanMilligan
@RyanMilligan

Has anyone seen an error like this before?

Uncaught DOMException: Failed to execute 'pushState' on 'History': A history state object with URL 'http:/#admin/posts' cannot be created in a document with origin 'http://localhost:8080' and URL 'http://localhost:8080/#admin/posts'.

It's coming from can-route-pushstate. I'm trying to figure out why it's happening, but if anyone knows where to look and can save me some time, that would be great. The router works fine in this application in development mode, but throws that error when using steal.production.js.
Matthew Phillips
@matthewp
@RyanMilligan you don't have a property named route on your viewModel do you?
RyanMilligan
@RyanMilligan
Hrm, a very interesting question. On the app view model? Not that I see.
Matthew Phillips
@matthewp
ok, that was a bit of a guess
I remember running into something like this and that was the problem
route is a semi-reserved word
RyanMilligan
@RyanMilligan
I just did a search for route and I don't see any properties with that name. Just references to can-route.
RyanMilligan
@RyanMilligan
We have app, page, slug, action, actionTwo and env.
RyanMilligan
@RyanMilligan
And because it only happens when we build, it's very difficult to diagnose.
Matthew Phillips
@matthewp
i saw something similar a while back and there was a loop occuring
ever time the route was set something in the route.data viewmodel was setting another property
I would check if something like that occurs
RyanMilligan
@RyanMilligan
Hrm. I have logs on all the setters for the viewmodel properties and they're only getting called the one time.
Matthew Phillips
@matthewp
hm, im out of ideas then
might have to debug within can-route
RyanMilligan
@RyanMilligan
I'll add some more logs around can-route and see what I can figure out, thanks.
RyanMilligan
@RyanMilligan
Ok, so I removed can-route-pushstate because I don't think we actually want it here, and that got rid of the error. But I still have an application that routes just fine when using steal.js and doesn't when using steal.production.js.
Matthew Phillips
@matthewp
The only thing I can suggest is to start removing stuff and then add it back
RyanMilligan
@RyanMilligan
I added some logging to setState, and both curParams and canRoute.data are just empty objects, which seems odd.
RyanMilligan
@RyanMilligan
It looks like the problem was caused by can-route-pushstate. It's working now, and I think the only reason it wasn't working when I removed it was because I had some other troubleshooting stuff hanging around that was getting in the way.
Chasen Le Hara
@chasenlehara
@RyanMilligan I’m glad you got it working. If you end up adding can-route-pushstate and running into the same issue, I’d be curious about what’s going on in these two places: 1) https://github.com/canjs/can-route-pushstate/blob/24dabe72b79acfcd30e516fc78599794d4f205b0/can-route-pushstate.js#L207 and 2) https://github.com/canjs/can-route-pushstate/blob/24dabe72b79acfcd30e516fc78599794d4f205b0/can-route-pushstate.js#L234
RyanMilligan
@RyanMilligan
I actually put a log statement in that second one, so I can tell you the three arguments being passed were null, null, http:/#admin/posts. But I wasn't able to debug up the chain very well to figure out where that weird URL was coming from because it was a built bundle.
Chasen Le Hara
@chasenlehara
Ok, gotcha. Very weird to see http:/#… how was only one slash added after the protocol 😅
nmingneau
@nmingneau
@justinbmeyer
Hi, i created an issue: canjs/can-fixture#160.
I also briefly thought about fixing it myself but i'm not sure why it needs to be _log since there is no use of underscore prefixes in de rest of the code.
Justin Meyer
@justinbmeyer

@/all We’re going to have a live stream right now about the next survey’s proposals:

https://forums.donejs.com/t/donejs-community-survey-november-2018/998

Live stream:

https://www.youtube.com/watch?v=kx-JxYSWlEc

@nmingneau there isn't in can-fixture, but there is in a lot of other canjs code
I'll write up a little more so you can do it if you'd like.
Chasen Le Hara
@chasenlehara
Our latest community survey is out! 📝 Check your email inbox for a link or learn more and take the survey from here: https://www.surveymonkey.com/r/donejs-survey-10
swipie
@swipie
Screenshot 2018-11-19 at 12.01.29.png
Since can@5.16.3 I get this warning inside my console when running my jest tests. But I’m not aware of any registered “not” helpers inside my code. Anyone an idea why this warning is triggerd?
Viktor Busko
@Lighttree

Hi guys, what "crossed" items means in this issue:

https://github.com/canjs/canjs/issues/4115#issuecomment-434448401

Mohamed Cherif Bouchelaghem
@cherifGsoul
@Lighttree The crossed packages means are working with IE11 and dont need to be fixed
Viktor Busko
@Lighttree
ah great
Kevin Phillips
@phillipskevin
@swipie it could just mean that two versions of can-stache-converters are being installed
Chasen Le Hara
@chasenlehara
@Lighttree We’re getting closer to having a stable version of can released with IE11 support. You might want to watch this PR too. 😊 canjs/canjs#4600
Viktor Busko
@Lighttree
Thanks !
Stef
@sronsiek
Is there a way to bind an event to all members of a DefineMap in a viewmodel? This should be simple (canjs 4.3)
'{viewModel} definemappram' function() { ... does not seem to fire
Justin Meyer
@justinbmeyer
@sronsiek what do you mean by "all members"?
You want to listen to when any property changes?
btw, if you are in 4.X, I'd suggest using connectedCallback() instead of the events object
You should be able to use canReflect.onPatches(): https://canjs.com/doc/can-reflect/observe.onPatches.html
Chasen Le Hara
@chasenlehara
IE11 is now included in the CanJS test suite. 🎉 Props to Kevin, Cherif, and others who contributed to this! 🏁 https://github.com/canjs/canjs/releases/tag/v5.17.0