Critiques pour ChromeLogger
ChromeLogger par BurningMoth
28 notes
- Noté 5 sur 5par Marc, il y a 7 mois
- Noté 5 sur 5par AmericanIntelligenceMachines, il y a un an
- Noté 5 sur 5par DP Dev, il y a 2 ans
- Noté 5 sur 5par Nina, il y a 3 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 17447093 de Firefox, il y a 4 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 16987925 de Firefox, il y a 4 ans
- Noté 5 sur 5par iNeRdS, il y a 5 ansi need you to monitor all my activities from this I.P. address I am sure you boys no what to do better than i describe
- Noté 5 sur 5par Don, il y a 6 ans
- Noté 4 sur 5par Utilisateur ou utilisatrice 15307722 de Firefox, il y a 6 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 15064500 de Firefox, il y a 6 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 14671107 de Firefox, il y a 7 ansIt unbreaks devtools. Thanks.
- Noté 5 sur 5par Utilisateur ou utilisatrice 14666722 de Firefox, il y a 7 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 13079274 de Firefox, il y a 7 ans
- Noté 5 sur 5par Utilisateur ou utilisatrice 14035565 de Firefox, il y a 7 ans
- Noté 5 sur 5par dadazord, il y a 8 ans
- Noté 5 sur 5par Joe Rooster, il y a 8 ansWorks as expected and better than other add-ons that (try to) put out messages from ChromePHP. Thank you :-)
- Noté 5 sur 5par Utilisateur ou utilisatrice 13483795 de Firefox, il y a 8 ansOk so the problems I reported earlier are fixed and logging is working as expected now. Thanks for a great bit of digital kit?
- Noté 5 sur 5par Patrick Brosset, il y a 8 ansWorks for me now. It was updated last week and it fixes the issue people have been talking about in the comments!
Thanks! - Noté 5 sur 5par Utilisateur ou utilisatrice 13328225 de Firefox, il y a 8 ansSince version 1.1.1 it works as it should. Thanks!
- Noté 4 sur 5par Utilisateur ou utilisatrice 13518149 de Firefox, il y a 8 ans
- Noté 5 sur 5par skrymir01, il y a 8 ansUpdated Review:
Issue's been resolved and logs my messages like a champ.
Previous Review:
Produces error: TypeError: window.console[method] is undefined
The reason appears to be that it assumes the console method will be sent in the response. This will be the case for the all logging types _except_ "log". According to the Chrome Logger technical specifications at https://craig.is/writing/chrome-logger/techspecs, if you want to log, you should send an empty string (to save on size). Therefore the add-on ought to use "log" if "method" is empty. I believe the following would be sufficient to fix it:
console[method || 'log']
I couldn't find a repository to log this issue at or submit a pull request, so I had to use the review section.