Skip to content
This repository has been archived by the owner on Dec 12, 2020. It is now read-only.

fix(catalog-tab): genre colors not appearing anymore #128

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

defvs
Copy link
Collaborator

@defvs defvs commented Nov 19, 2020

MCatalog changed a few things in the sheet so genre colors were broken:

  • Deleted Subgenre column (why...)
  • Renamed Genre to Label (why???)
  • Label is actually a very broad genre; no more subgenres like Neurofunk or Liquid DnB, only Drum and Bass now
  • Removed brand column (Instinct / Uncaged)

I don't feel good about these changes but well it's their thing.

This fixes the genre colors not being applied because the column is now named "Label" instead. I also cleaned up the column list to get rid of the "column not found" warning.
Had to add exceptions to the genreColors map because your implementation of RoughMap does a search by checking if the input string is contained in any of the keys; this caused issues with "EP" since it's contained within "Drumstep".
Also cleaned up the code a tiny bit

@defvs defvs requested a review from xeruf November 19, 2020 18:11
@defvs defvs self-assigned this Nov 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant