altsheets
altsheets
Interesting. Please keep us posted about what you find out.
I have repaired that, see my fork: [altsheets /lib/CoinFactory.js#L32](https://github.com/altsheets/CoinMarketCapAPI/blob/f728da2baa08d20171589fa913af230356a0f27f/lib/CoinFactory.js#L32)
Please keep on updating your code. We have been very happy users. Eventually actually ran our own instance of your code on our own server. But now, that started crashing...
Thx a lot! Good to hear from you. About my pull requests: Sure. When I make improvements, I make them public, so others can benefit too, no problem at all....
As far as I can see, lines 25 [CoinFactory.js:25:64](https://github.com/Nexuist/CoinMarketCapAPI/blob/master/lib/CoinFactory.js#L25): ``` "symbol": tds.eq(4).text().match(/\s([a-z]|[0-9]|@)+\s/i)[0].trim().toLowerCase(), ``` and column 64 would point to a failing array index [0], so I guess tds.eq(4).text().match(/\s([a-z]|[0-9]|@)+\s/i) is empty?...
I have repaired that, see my fork: [altsheets ... /lib/CoinFactory.js#L30](https://github.com/altsheets/CoinMarketCapAPI/blob/f728da2baa08d20171589fa913af230356a0f27f/lib/CoinFactory.js#L30) the symbol actually (now?) also appears in the second column.
Quite a bit of the CMC page layout must have been changed recently, so several things needed repair. I have fixed what I could find, but I am not 1000%...