Дмитрий

map это функция, она должна быть вызвана, вы же делаете ей присвоение. К тому же если она ничего не возвращает, использовать ее бессмысленно, используйте вместо нее forEach

strings.forEach(element => {
  console.log(element);

  let sel = element.querySelector('.selector');
  if (sel.value === '0') {
    error [element.name] = sel.name;
  }
});

Большинство программистов, в том числе те, кто будет вас собеседовать - как мартышки, им сказали что правильно так и так, они не задумываясь передают это дальше и бьют по голове каждого, кто осмеливается делать не по шаблону. Вся эта возня с экшинами и типами безусловно избыточна в редаксе в 95% случаев. Я вообще делал надстройку, чтобы напрямую вызывались процедуры, изменяющие стейт. Хотя не рискнул бы выложить это в качестве портфолио, по упомянутым выше причинам.

В самом вашем коде ничего плохого нет, вы можете оставить его, в комментариях пояснив, что вынос экшинов в отдельный файл в данном случае ведет к избыточному бойлерплейту. Единственное все же нагляднее, если mapStateToProps и MapDispatchToProps вынесены на верх, а не вставлены в коннект непосредственно.

Не вижу смысла выделять city ни в ключ, ни в отдельный параметр. В чем смысл? Сделайте плоскую таблицу

this.cities1 = [
        { city: 'Москва', cityCode: 'moskva54', views: '53М', visites: '1,1M', isActive: true  },
        { city: 'Владимир', cityCode: 'vladimir', views: '23М', visites: '1,2M', isActive: false }
];

Либо вообще вместо массива объект:

this.cities1 = {
  'Москва': { city: 'Москва', cityCode: 'moskva54', views: '53М', visites: '1,1M', isActive: true  },
  'Владимир': { city: 'Владимир', cityCode: 'vladimir', views: '23М', visites: '1,2M', isActive: false }
};

В этом случае вы моментально получаете данные по городу или по нескольким, так как названия индексированы.. Если же вам нужен список, вы просто получаете Object.keys(this.cities1) или Object.values(this.cities1) если вместо со всеми данными

Если корректировать ваш вариант, то нужно спредить получаемый из func результат

if (typeof elem == 'object') {
      res.push(...func(elem));
    } else {
      res.push(elem);
    }

но есть еще функция flat у массивов, можно использовать ее

elem.flat(Infinity)

document.querySelectorAll(".m1")
  .forEach(el => el.addEventListener("click",  function(e){ 
    this.style.backgroundColor = "red";
  }));

Потому что принято записывать значения 8-битовыми порциями, просто соглашение

Переименуйте переменную внутреннего цикла а лучше пользуйтесь современным синтаксисом const или let

Я когда учился, делал универсальную таблицу с возможностью группировки, отбора, сортировки строк, подсчетом итоговых значений. Это очень немаленький проект и хорошо прокачивает. Естественно без использования готовых ui компонентов

Потому что каждый раз при нажатии на первую кнопку добавляется listener. Он не перезаписывается. Соответственно при нажатии на вторую кнопку срабатывают все ранее подключенные нажатием на первую кнопку listener-ы. И поскольку переменная a объявлена внутри функции, то увеличиваются все a внутри всех ранее подключенных слушателей.

Анализируя атрибут name

const maxNumber = Math.max(...Array
  .from(document.querySelectorAll('input.form-control.quantity'))
  .map(el => Number(el.name.split(/\[|\]/g)[1] || 0))
)

Очевидно, что для какого-то элемента строчка
const member = guild.members.get(i);

возвращает undefined

Поставьте брейкпойнт с условием в этом месте и посмотрите на каком элементе. Очевидно что где-то у guild.members элементов меньше чем в массиве bot.muted

Потому что возвращается коллекция, из которой динамически удаляются данные когда элемент больше не соответствует выставленному отбору. Зафиксируйте значения в массиве, тогда будет работать.

var str = Array.from(document.getElementsByClassName('test'))

function getIntervals (arr) {
   const mlsInterval = 30 * 60 * 1000

   const openHours = arr.filter(v => v.type === 'open').flatMap(v => {
      const [d1, d2] = [new Date(v.from).getTime(), new Date(v.to).getTime()]
      const length = Math.floor(( d2 - d1 ) / mlsInterval)
      const hours = Array.from({ length }, (z, ind) => new Date(d1 + ind * mlsInterval))
      return hours
   })

   const closeIntervals = arr.filter(v => v.type === 'closed').map(v => 
     [new Date(v.from).getTime(), new Date(v.to).getTime()]       
   )

   const result = openHours.filter(v =>  
     !closeIntervals.some(([start, stop]) => start <= v && stop > v)
   ).map(v => new Date(v).toString().match(/\d\d:\d\d:\d\d/g)[0])

   return result
}

Array.from(document.querySelectorAll('input')).every(v => v.value)