Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[Embed] Remove Console Logs #2760

Closed
kfei opened this issue Aug 2, 2015 · 1 comment
Closed

[Embed] Remove Console Logs #2760

kfei opened this issue Aug 2, 2015 · 1 comment
Labels
Confirmed Bug
Milestone
2.1

Comments

@kfei
Copy link

kfei commented Aug 2, 2015

There are four console.log expressions in embed.js. Is it an expected behaviour?
@jlukic
Copy link
Member

jlukic commented Aug 3, 2015

bad bad bad bad me, 3 are wrong

If you include site.js you can temporarily disable all console functionality in production using $.site('disable console'); (hidden feature).

@jlukic jlukic added this to the 2.1 milestone Aug 3, 2015
@jlukic jlukic changed the title console.log in embed.js [Embed] Remove Console Logs Aug 3, 2015
jlukic added a commit that referenced this issue Aug 10, 2015
@jlukic
#2760 - remove console.log from embed
145b4f5
@jlukic jlukic closed this as completed Aug 10, 2015
jlukic added a commit that referenced this issue Aug 10, 2015
@jlukic
#2758 #2760 fix this context for transition callbacks
0d23f97
@aexvir aexvir mentioned this issue Mar 29, 2024
Open
@CharaD7 CharaD7 mentioned this issue Mar 29, 2024
Open
@machine-snyk-provi machine-snyk-provi mentioned this issue Mar 29, 2024
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
Confirmed Bug
Projects
None yet
Milestone
2.1
Development

No branches or pull requests
2 participants
@jlukic @kfei