Skip to content

Conversation

@ash-t-luu
Copy link
Contributor

@ash-t-luu ash-t-luu commented Feb 3, 2024

Overview

Issue Type

  • Bug
  • Feature
  • Tech Debt

Description
Working tests for #11-14. Modified config files and package.json to be ready for launch. Merged with updated code.

Ticket Item
RL-65: Add functionality to getComponentType to parse ExportDefaultDeclaration functions/components
RL-66: Add functionality to getComponentType to parse FunctionDeclaration
RL-56: TESTS #11-14

Expected behavior
Working tests for 11-14, updated parser logic for determining component type, changed showError to showNotification upon no file upload, updated package.json and other configs to be ready for production.

… updated babel/parser, took out comments in test files, added paths to exclude complie in tsconfig, note: test 14 is not fully working
…ls, to allow programmatic dissmisal of modal if no file was selected
…ded my testing suite to this branch and files, all of the tests in 14 works except the component1. not sure why
…found based on the type. all the tests in tc_14 are now passing.
…testing for #11-14, changed logic in parser to account for all function declarations
Copy link
Contributor

@Johnny-Arroyo Johnny-Arroyo left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@Ponch49 Ponch49 left a comment

Choose a reason for hiding this comment

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

lgtm

@Ponch49 Ponch49 merged commit ffeb6ff into dev Feb 3, 2024
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.

5 participants