Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 17 20:12

    matthewp on master

    Update dist for release (compare)

  • Oct 17 20:12

    matthewp on v6.1.2

    Update dist for release 6.1.2 (compare)

  • Oct 17 20:04

    matthewp on master

    Update dist for release (compare)

  • Oct 17 20:04

    matthewp on v6.1.2-0

    Update dist for release 6.1.2-0 (compare)

  • Oct 17 19:17
    akbaruddin starred canjs/canjs
  • Oct 17 17:25
    matthewp edited #5384
  • Oct 17 17:00

    matthewp on can-dom-mutate-2.0.7

    (compare)

  • Oct 17 17:00

    matthewp on master

    fix(package): update can-dom-mu… Merge pull request #5386 from c… (compare)

  • Oct 17 17:00
    matthewp closed #5386
  • Oct 17 16:27
    greenkeeper[bot] labeled #5386
  • Oct 17 16:27
    greenkeeper[bot] opened #5386
  • Oct 17 16:27

    greenkeeper[bot] on can-dom-mutate-2.0.7

    fix(package): update can-dom-mu… (compare)

  • Oct 17 16:25

    matthewp on can-dom-mutate-2.0.6

    (compare)

  • Oct 17 16:24
    matthewp closed #5379
  • Oct 17 16:24
    matthewp commented #5379
  • Oct 17 15:26

    greenkeeper[bot] on can-type-1.1.2

    (compare)

  • Oct 17 15:26

    matthewp on master

    fix(package): update can-type t… Merge pull request #5385 from c… (compare)

  • Oct 17 15:26
    matthewp closed #5385
  • Oct 17 14:11
    matthewp closed #5377
  • Oct 17 14:11
    matthewp closed #5376
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
cc @Lighttree 😃
Chasen Le Hara
@chasenlehara
@/all Today’s the last day to vote in our community survey, please fill it out here if you haven’t already: https://www.surveymonkey.com/r/donejs-survey-10
Stef
@sronsiek
@justinbmeyer: Thank you - exactly what I wanted - any change to any DefineMap property now triggers a handler. Is there a need / way to de-register this when leaving the page?
Yura Ostroukh
@iSokrat
Hello everyone! Does somebody have an experience with migration CanJs 2 to CanJs 3? Are there some bottlenecks with this process not described into https://canjs.com/doc/migrate-3.html? Thank you for any help!
Viktor Busko
@Lighttree
@chasenlehara :D thanks, already created story for migration.