search

publiclab / infragram

A minimal core of the Infragram.org project in JavaScript
https://infragram.org/sandbox/
GNU General Public License v2.0
44 stars 166 forks source link

Cleaning of unecessary logs #289

Closed shubhangi013 closed 2 years ago

shubhangi013 commented 2 years ago

There are a lot of very unnecessary logs throughout the code. For example :

https://github.com/publiclab/infragram/blob/ed49410ea322a4e09f0b670efa5d606a0f9cf57b/dist/infragram.js#L487

and https://github.com/publiclab/infragram/blob/ed49410ea322a4e09f0b670efa5d606a0f9cf57b/dist/infragram.js#L1497

Can we remove these? there are other helpful ones that actually help the developers while contributing to the code like: https://github.com/publiclab/infragram/blob/ed49410ea322a4e09f0b670efa5d606a0f9cf57b/dist/infragram.js#L113

however, as pointed out above, there are some unnecessary and vague ones too. I would like to work and remove/ rephrase these.

TildaDares commented 2 years ago

Hi @shubhangi013, go ahead. Thanks!

govindgoel commented 2 years ago

@shubhangi013 yeah, you can remove the redundant logs and maybe comment out those which might help in debugging something.

shubhangi013 commented 2 years ago

@shubhangi013 yeah, you can remove the redundant logs and maybe comment out those which might help in debugging something.

Right! Thankyou 😊

TildaDares commented 2 years ago

Resolved in #293