Rsa97

Traditionally, and according to ECMAScript, all primitives and objects are loosely unequal to undefined and null

https://developer.mozilla.org/en-US/docs/Web/JavaS...

const USER = {VALUTE: 'usd'};
DATA.market_data.current_price[USER.VALUTE]

if (target.closest('.category-button')) {
  ...
}

https://developer.mozilla.org/ru/docs/Web/API/Elem...

На слабенькую троечку.
Глобальные переменные, рекурсия без необходимости, тернарный оператор вместо if, работа со стилями вместо классов, переназначение onclick вместо addEventListener со всплытием. Это только то, что с ходу в глаза бросилось.
Ну и общее впечатление отсутствия единого стиля, ощущение, что куски понадёрганы из разных примеров.

.filter всегда возвращает массив, пусть и пустой, а ![] === false
Используйте .some

document.querySelectorAll('.bottomONE, .bottomTWO');

В переменную g у вас записывается ссылка на функцию, которая вернулась из f(). Пока эта ссылка доступна из глобального контекста (через g), функция сохраняется в памяти. Весь контекст, связанный с функцией, также сохраняется, поскольку тоже доступен из глобального контекста (через g и функцию).
Присваивая переменной null вы, тем самым, делаете функцию и связанный с ней контекст недоступными из глобального контекста. Значит при следующем проходе сборщик мусора удалит их как неиспользуемые.

function round45(value) {
  return Math.round(value / 45) * 45;
}
round45(90.3222341); // 90
round45(136.99); // 135
round45(358); // 360
round45(46); // 45

let diff = (new Date('01 Dec 2022') - new Date()) / 1000;
const seconds = diff % 60;
diff = (diff - seconds) / 60;
const minutes = diff % 60;
diff = (diff - minutes) / 60;
const hours = diff % 24;
const days = (diff - hours) / 24;
console.log(days, hours, minutes, seconds);
// 587 6 30 21.51799999922514

document
  .getElementsByTagName('iframe')[0]
  .contentWindow
  .document
  .getElementsByTagName('iframe')[0]
  .contentWindow
  .document
  .getElementsByClassName('it_this')[0]

Но если какой-то из iframe не на вашем домене, то будет применена политика Same Origin Policy и доступ к содержимому вы можете не получить.

Вы забыли про асинхронность. К тому моменту, как приходят данные с сервера и выполняется .then, строка checkKey('cities', cityTo); уже давно отработала и ничего не нашла.
Самое простое - добавить ещё один вызов checkKey в блоке .then после setItem.