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

Drop Arborist UI support PEDS-161 #152

Merged
merged 38 commits into from
May 20, 2021
Merged

Conversation

bobaekang
Copy link

@bobaekang bobaekang commented May 20, 2021

Ticket: PEDS-161

This PR removes the support for "Arborist UI" from the portal, which is not relevant to the PCDC use case. In a sense, this PR can be considered as reverting uc-cdis#592 -- with additional refactoring to affected files where relevant. The refactoring effort mostly involves converting class components to function components and adding proper JSDoc comment to components.

This PR also removes <AccessTable> component that is no longer in use since #41.

bobaekang added 30 commits May 19, 2021 09:45
+ minor refactoring in rendering items
+ jsdoc & minor refactoring
+ Import components directly instead of receiving as props
@bobaekang bobaekang changed the title Drop arborist UI support PEDS-161 Drop Arborist UI support PEDS-161 May 20, 2021
@bobaekang bobaekang merged commit 0f47d9c into pcdc_dev May 20, 2021
@bobaekang bobaekang deleted the feat/drop-arborist-ui-support branch May 20, 2021 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant