Logging capability about aura HOT 8 CLOSED

@KMK1986 Hadn't thought about this but I dig it. Do you want to send a pull request so we can have a look at the code?

from aura.

@dustinboston // i will try tonight or whenever I get time. I think I need to learn how to send a pull request first :)

Thank you for the quick reply!

from aura.

@KMK1986 Pull requests are relatively straight-forward even if they seem a bit of a challenge at first :) Once you've made a fork of this repo on GitHub (which you can do by navigating to the main repo page and clicking 'fork') you'll get your own copy of the project. You can then clone that repo locally using git clone <repo URL>, make your changes, commit them and then push back to GitHub. Once thats done, go to the repo page for your fork of the project and click on the button that says Pull Request. You should be able to make a PR from there :)

If you need any help this might help https://help.github.com/articles/using-pull-requests

from aura.

@addyosmani // Thank you for the info! I will try to do it soon.

from aura.

Hello,

Sorry for the delay. Been really busy..

I actually created a test page on a repo.

https://github.com/kmk1986/test.git

you need open it with chrome then open inspector. Click each links one by one. You will see something like "["Module1: published event hello to Module2 with event data ", "module1"]"

I think you will get the idea once you open the page..

Please don't look at js files :) it's just proof of concepts.

from aura.

On this, I think if it were implemented as an extension I wouldn't mind us including it as an optional component, but otherwise it might get closed as patchwelcome. Thoughts @KMK1986?

from aura.

@addyosmani // Yes, sure! I started using Backbone Aura for my projects recently. Once I get familiar with it, I will try to create an extension. Thank you for your opinion!

from aura.

Thanks for the discussion! Closing due to age of ticket and applicability to the refactor. Please feel free to reopen if you would like to address this via a pull request or specific solution with the current version in master.

btw, logging is being added via a recent PR.

from aura.