topcoat-data / topcoat-public

TopCoat components, visualizations and themes
MIT License
1 stars 3 forks source link

fix: remove reference to headerClasses prop #246

Closed sareg0 closed 1 year ago

sareg0 commented 1 year ago

What this does

@benbarnett picked up on an error with the TTable that was created with the removal of headerClasses in #239. See screenshots for the error

Notes for the reviewer

Go to https://snyk-insights.topcoatdata.app/ Change the branch in config.yml that points to topcoat-public to the branch name of this PR,as below:

modules:
    - git: https://github.com/topcoat-data/topcoat-public.git
      revision: fix/headClasses-prop-still-being-referenced

Save the config.yml file, and click the 'Build and Sync modules' button (it's a cloud symbol)

Screenshot 2023-06-02 at 17 28 12

Refresh the reports in https://snyk-insights.topcoatdata.app/

Test the table on Vulnerabilities Detail Report. Make sure it displays and each section you open is also displaying.

More information

Screenshots / GIFs

Before

Screenshot 2023-06-14 at 16 02 05

After

Screenshot 2023-06-14 at 15 58 27