RTIInternational / SMART

Smarter Manual Annotation for Resource-constrained collection of Training data
MIT License
220 stars 31 forks source link

Fix broken label guide #279

Closed AstridKery closed 11 months ago

AstridKery commented 11 months ago

The recent table upgrades swapped some of the React components from class-based to functional. However, since not everything was swapped over we ended up with some hiccups. The CodebookLabelMenu is still a class-based component which relies on a getLabel action that was removed when the useLabels hook was created. Trouble is that the class-based components can't use the hook, and so still need the old action. Adding back in the action code fixed the issue.

Long-term we will want everything to be moved over to functional components so we don't have this sort of duplication. However, I don't recommend we try and do that right now as we want to deploy recent changes in the near term.