Skip to content

Commit

Permalink
Merge pull request #126 from Financial-Times/remove_nLogger
Browse files Browse the repository at this point in the history
remove n-logger use console.error
  • Loading branch information
danieleruiz committed Sep 14, 2023
2 parents 43ece6f + 78597c6 commit a727c8d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions index.js
Expand Up @@ -6,7 +6,6 @@ const surveyBuilder = require('./src/survey-builder');
const postResponse = require('./src/post-response');
const getAdditionalInfo = require('./src/get-additional-info');
const dictionary = require('./src/dictionary');
const logger = require('@financial-times/n-logger').default;


function getSurveyData(surveyId) {
Expand Down Expand Up @@ -180,7 +179,8 @@ module.exports.init = (appInfo = {}) => {

const container = document.querySelector(`${containerSelector} .n-feedback__container`);
if (!container) {
logger.error('The container was not found on the page.')
// eslint-disable-next-line no-console
console.error('The container was not found on the page.')
throw new Error('The container was not found on the page.');
}
container.classList.remove('n-feedback--hidden');
Expand Down

0 comments on commit a727c8d

Please sign in to comment.