Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Revert "DEV: Log deprecated FA 4.7 icon names in /logs" 

This reverts commit 2dd9727c11.
This commit is contained in:
Penar Musaraj 2019-12-18 12:44:52 -05:00
parent 2dd9727c11
commit 274c22e7a2
2 changed files with 7 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/assets/javascripts/discourse-common/lib/icon-library.js.es6
View File

@ -1,5 +1,4 @@
import { h } from "virtual-dom"; import { h } from "virtual-dom";
import { reportToLogster } from "discourse/lib/utilities";
import attributeHook from "discourse-common/lib/attribute-hook"; import attributeHook from "discourse-common/lib/attribute-hook";
import deprecated from "discourse-common/lib/deprecated"; import deprecated from "discourse-common/lib/deprecated";
@ -585,11 +584,6 @@ function warnIfDeprecated(oldId, newId) {
deprecated( deprecated(
`Please replace all occurrences of "${oldId}"" with "${newId}". FontAwesome 4.7 icon names are now deprecated and will be removed in the next release.` `Please replace all occurrences of "${oldId}"" with "${newId}". FontAwesome 4.7 icon names are now deprecated and will be removed in the next release.`
); );
const errorData = {
message: `FA icon deprecation: replace "${oldId}"" with "${newId}".`,
stacktrace: Error().stack
};
reportToLogster(errorData);
} }
function handleIconId(icon) { function handleIconId(icon) {

13
app/assets/javascripts/discourse/lib/utilities.js.es6
View File

@ -376,7 +376,12 @@ export function postRNWebviewMessage(prop, value) {
} }
} }
export function reportToLogster(data) { function reportToLogster(name, error) {
const data = {
message: `${name} theme/component is throwing errors`,
stacktrace: error.stack
};
Ember.$.ajax(`${Discourse.BaseUri}/logs/report_js_error`, { Ember.$.ajax(`${Discourse.BaseUri}/logs/report_js_error`, {
data, data,
type: "POST", type: "POST",
@ -387,11 +392,7 @@ export function reportToLogster(data) {
export function rescueThemeError(name, error, api) { export function rescueThemeError(name, error, api) {
/* eslint-disable-next-line no-console */ /* eslint-disable-next-line no-console */
console.error(`"${name}" error:`, error); console.error(`"${name}" error:`, error);
const errorData = { reportToLogster(name, error);
message: `${name} theme/component is throwing errors`,
stacktrace: error.stack
};
reportToLogster(errorData);
const currentUser = api.getCurrentUser(); const currentUser = api.getCurrentUser();
if (!currentUser || !currentUser.admin) { if (!currentUser || !currentUser.admin) {