Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/vadc data dictionary table #1471

Merged

Conversation

jarvisraymond-uchicago
Copy link
Contributor

@jarvisraymond-uchicago jarvisraymond-uchicago commented Jan 10, 2024

Jira Ticket: VADC-828

Notes

  • Adds Mantine V6, does not use AntD per discussion with @craigrbarnes to make this easier to port over frontend framework
  • Includes start of implementation of dropdown, search, sort and pagination. Completion of that functionality will occur in subsequent PRs associated with their own JIRA tickets (they work but there's numerous edge cases that need to be addressed so they work together). Charts and table styling will occur in subsequent tickets.
  • Renames Application to Atlas Data Dictionary

output

New Features

  • Adds table with styling to VADC Atlas Data Dictionary App

Improvements

  • Fixes stylelint issue in src/Analysis/SharedUtils/TeamProject/TeamProjectHeader/TeamProjectHeader.css

Dependency updates

  • Adds Mantine V6 to VADC

tianj7 and others added 24 commits November 28, 2023 15:10
…d button for dropdown and columns per design
…earch bar control buttons keyboard accessible, changed location of pagination controls, resolved issue with typescript linting and pagination controls
…hlighting to function for hidden value summary div
… and recreated test data with unique conceptID values. Added updated icons
…rated out detail tables into seperate components, added preview chart component
@jarvisraymond-uchicago jarvisraymond-uchicago changed the base branch from master to feat/vadc_sprint25 January 10, 2024 18:17
Base automatically changed from feat/vadc_sprint25 to master January 10, 2024 19:48
@jarvisraymond-uchicago jarvisraymond-uchicago changed the base branch from master to feat/vadc_sprint01 January 10, 2024 19:53
@jarvisraymond-uchicago jarvisraymond-uchicago marked this pull request as ready for review January 17, 2024 20:29
@pieterlukasse
Copy link
Contributor

@jarvisraymond-uchicago can you please also rename your new DataDictionary folder (and files, class names, etc) to AtlasDataDictionary? This will help us distinguish it from the existing Gen3 DataDictionary that already exists in this same code base (see src/DataDictionary/).

@jarvisraymond-uchicago
Copy link
Contributor Author

@jarvisraymond-uchicago can you please also rename your new DataDictionary folder (and files, class names, etc) to AtlasDataDictionary? This will help us distinguish it from the existing Gen3 DataDictionary that already exists in this same code base (see src/DataDictionary/).

Implemented as requested.

Copy link
Contributor

@pieterlukasse pieterlukasse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks. Great start!

@jarvisraymond-uchicago jarvisraymond-uchicago merged commit 8775b75 into feat/vadc_sprint01 Jan 23, 2024
4 checks passed
@jarvisraymond-uchicago jarvisraymond-uchicago deleted the feat/vadcDataDictionaryTable branch January 23, 2024 21:17
jarvisraymond-uchicago added a commit that referenced this pull request Jan 31, 2024
* Remove unnecessary outcome prop from histogram rendering (#1459)

* feat(dataDictionaryAppPlaceholder): Initial commit

* feat(dataDictionaryAppPlaceholder): Ran linter

* feat(vadcDataDictionaryTable): Initial commit

* feat(vadcDataDictionaryTable): created working table header abstraction

* feat(vadcDataDictionaryTable): Wrote styles for tableHeaders and removed unneeded columns

* feat(vadcDataDictionaryTable): added entries header

* feat(vadcDataDictionaryTable): Updated component sort logic

* feat(vadcDataDictionaryTable): Simplified functions for updating arrows CSS classes

* feat(vadcDataDictionaryTable): added secondary row to hold data viz and additional information

* feat(vadcDataDictionaryTable): Added search

* feat(vadcDataDictionaryTable): Added search highlighting

* feat(vadcDataDictionaryTable): Added search highlighting for hidden items

* feat(vadcDataDictionaryTable): added pagination controls:

* feat(vadcDataDictionaryTable): Began converting JSX to TSX

* feat(vadcDataDictionaryTable): Converted remaining JSX to TSX, changed button for dropdown and columns per design

* feat(vadcDataDictionaryTable): Styled and updated expandable row

* feat(vadcDataDictionaryTable): fixed length of highlight for expanded chart

* feat(vadcDataDictionaryTable): Changed location of search bar, made search bar control buttons keyboard accessible, changed location of pagination controls, resolved issue with typescript linting and pagination controls

* feat(vadcDataDictionaryTable): Resolved ESLINT issues, got search highlighting to function for hidden value summary div

* feat(vadcDataDictionaryTable): added preprocessing function to add percentages to JSON

* feat(vadcDataDictionaryTable): Updated pagination logic, search logic and recreated test data with unique conceptID values. Added updated icons

* feat(vadcDataDictionaryTable): Fixed animation for dropdown row, seperated out detail tables into seperate components, added preview chart component

* feat(vadcDataDictionaryTable): Updated GenerateTableData

* feat(vadcDataDictionaryTable): Removed console log statements

* vadcDataDictionaryTable): removed console log

* feat(vadcDataDictionaryTable): made css more consistent

* feat(vadcDataDictionaryTable): Fixed bug in GenerateTableData, fixed style lint issues in DataDictionary.css and TeamProjectHeader.css

* feat(vadcDataDictionaryTable): Abstracted function checkIfCellContainsSearchTerm into CheckSearchTermUtils to better organize and increase testability

* feat(vadcDataDictionaryTable): Abstracted out remaining CheckSearchTermUtils

* feat(vadcDataDictionaryTable): Wrote unit test for detail tables

* feat(vadcDataDictionaryTable): added test for table row text

* feat(vadcDataDictionaryTable): fixed linting issues

* feat(vadcDataDictionaryTable): Wrote unit tests for CheckSearchTermUtils

* feat(vadcDataDictionaryTable): Added tests for PreprocessTableData and SearchBar

* feat(vadcDataDictionaryTable): Linted test files

* feat(vadcDataDictionaryTable): Added working pagination controls test

* feat(vadcDataDictionaryTable): linted test file

* feat(vadcDataDictionaryTable): wrote unit test for entries header

* feat(vadcDataDictionaryTable): Added working test for Header component

* feat(vadcDataDictionaryTable): added test for ColumnHeaders

* feat(vadcDataDictionaryTable): fixed test issues and eslint issues

* feat(vadcDataDictionaryTable): removed unneeded comments and formated code

* feat(vadcDataDictionaryTable): fixed off by one error in entries header props

* feat(vadcDataDictionaryTable): ran linter

* feat(vadcDataDictionaryTable): Reduced test data size, added explanatory comment for creating test data with larger sizes

* feat(vadcDataDictionaryTable): Ran linter

* feat(vadcDataDictionaryTable): removed unintentionally changed file

* feat(vadcDataDictionaryTable): Renamed folder to AtlasDataDictionary

* feat(vadcDataDictionaryTable): changed references from data dictionary to atlas data dictionary

* feat(vadcDataDictionaryTable): changed more  references from data dictionary to atlas data dictionary

* feat(vadcDataDictionaryTable): Removed unneeded console log statement

* feat(vadcDataDictionaryTable): changed 2 more references from data dictionary to atlas data dictionary

---------

Co-authored-by: Jian <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants