Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

amp-list: [src] changes don't properly dispose old bindings

See original GitHub issue

I am currently facing a Maximum number of bindings reached (2000). Additional elements with bindings will be ignored issue.

based on my research, at [bind-impl.js#L532] (https://github.com/ampproject/amphtml/blob/master/extensions/amp-bind/0.1/bind-impl.js#L532) will change the limitExceeded from false to true, and this causes me an error which is = 106 + 1 > 106 when i am trying to select countries on my 4th or 5th times. ss1 ss2 ss3

can you please elaborate more on the L530, what is this IF statement checking bindings.length + boundProperties.length > limit does ? I am also curious how does this IF check is relevant to 2000 bindings numbers.

Appreciate more sharing from you. Thanks in advanced. @choumx

Issue Analytics

State:
Created 6 years ago
Comments:20 (6 by maintainers)

Top GitHub Comments

1reaction

dreamofabearcommented, Jan 9, 2018

@leonalicious Verified this is fixed on version 1515455265699.

1reaction

dreamofabearcommented, Jan 6, 2018

Stumbled on a couple more bugs. 😄

Duplicate rescan due to AmpEvents.DOM_UPDATE event.
Duplicate amp-list render/fetch due to race between mutatedAttributesCallback and layoutCallback.