Merge-Reste entfernen und Typpruefung ins Tor vor dem Image
v0.6.0-rc.2 brach in Produktion die Raumliste: RoomListItemViewModel.ts rief
SettingsStore.getValue("feature_room_list_sections") auf einen Schalter, den
Upstream entfernt hat - Sektionen laufen dort ueber RoomList.showSections. Der
Merge hatte ueberall Upstreams Seite genommen (Menue, View, Snapshot, Typen),
nur diese eine const-Zeile blieb stehen. Sie wurde tree-weit von niemandem
gelesen und warf bei jedem Raumlisteneintrag.
Zweiter Rest gleicher Art: der Import ElementDesktopLogoSvg in SdkConfig.ts.
Upstream nutzt ihn in desktopBuilds.logo, wir ueberschreiben den Block bewusst
mit eigenem Logo und Release-Pfad - der Import blieb ungenutzt zurueck. Das
Fork-Patch selbst ist unveraendert.
Abgesichert: alle 135 abgefragten gegen die 152 registrierten Einstellungen im
Quellbaum verglichen - keine weiteren Leichen.
Neuer Job typecheck, den docker_web als needs fuehrt. Der web-Job baut nur;
webpack entfernt Typen, ohne sie zu pruefen. tsc meldete beide Reste (TS2345,
TS6133) die ganze Zeit - gefragt hatte ihn niemand.
Massstab ist "kein Fehler ausserhalb von node_modules", weil Upstream v1.12.26
selbst nicht typrein ist: matrix-js-sdk 42.2.0 wirft drei Fehler in der eigenen
Quelle. In einem sauberen v1.12.26-Checkout nachgeprueft - dieselben drei, sonst
nichts.
Das Tor ist in beide Richtungen geprueft: mit wieder eingesetzter Zeile
scheitert es und benennt beide Fehler, ohne sie besteht es. Farbcodes werden
vorher entfernt - nx faerbt auch in der Pipe, zwischen "error" und "TS" steht
sonst eine Escape-Sequenz und das grep liefe ins Leere. Zusaetzlich scheitert
der Job bei leerer tsc-Ausgabe, damit ein stiller Erfolg nicht als Pruefung
durchgeht.
This commit is contained in:
@@ -13,7 +13,6 @@ import { type DeepReadonly } from "shared-types";
|
||||
import { SnakedObject } from "./utils/SnakedObject";
|
||||
import { type IConfigOptions, type ConfigOptions } from "./IConfigOptions";
|
||||
import { isObject, objectClone } from "./utils/objects";
|
||||
import ElementDesktopLogoSvg from "../res/img/element-desktop-logo.svg";
|
||||
|
||||
// see element-web config.md for docs, or the IConfigOptions interface for dev docs
|
||||
export const DEFAULTS = {
|
||||
|
||||
@@ -396,8 +396,6 @@ export class RoomListItemViewModel
|
||||
})
|
||||
: undefined;
|
||||
|
||||
const canMoveToSection = SettingsStore.getValue("feature_room_list_sections");
|
||||
|
||||
// Build sections list for the "Move to section" submenu
|
||||
const sections: Section[] = RoomListItemViewModel.buildSections(roomTags, availableSections);
|
||||
const areSectionsEnabled = SettingsStore.getValue("RoomList.showSections");
|
||||
|
||||
Reference in New Issue
Block a user