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

Added FileInput component #343

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

Conversation

fung-csf
Copy link

@fung-csf fung-csf commented Aug 1, 2021

The component is able to display the name of the file selected by user.

File Upload component that display the name of the file selected by user.
added the line:

 import { FileInput } from './FileInput';

Edited the line:

 export { UploadProps, FileInput };
@vercel
Copy link

vercel bot commented Aug 1, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/react-component/upload/J4mJTeXnhoNxcj7nyW5pYFTY2vJX
✅ Preview: https://upload-git-fork-fung-csf-master-react-component.vercel.app

fung-csf and others added 22 commits August 24, 2021 21:12
Change function parameters for FileInput.js  for integration with UploadAlt.js
added   "antd": "^4.16.13"   as dependency
import UploadAlt
Create UploadAlt.js
Added new style to FileInput.js
In order to change extension from .js to .tsx
create FileInput with .tsx extension
style formatting to conform to linting requirements
change function name from inputFileValue to FileContent
Change UseState Variable name from "input" to "content"
 Update UseState variable "content" to  "filePresent"
Remove  braces in file import
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 1 alert when merging 8bafddb into 77f44b8 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

added the rules:

 "no-shadow": "off",
  "@typescript-eslint/no-shadow": ["error"]
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 1 alert when merging 77ccbb1 into 77f44b8 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

remove base rules in eslintrc.js
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 1 alert when merging 3f26eae into 77f44b8 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

remove   line :

...base,
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 2 alerts when merging bc17a1a into 77f44b8 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

added new rules
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 2 alerts when merging 40c0a90 into 77f44b8 - view on LGTM.com

new alerts:

  • 1 for Overwritten property
  • 1 for Unused variable, import, function or class


 "@typescript-eslint/no-shadow": 0,
     "no-shadow": 0,
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 1 alert when merging 3c9a782 into 77f44b8 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 7 alerts when merging 9340863 into 77f44b8 - view on LGTM.com

new alerts:

  • 7 for Unused variable, import, function or class

jackisgood13 and others added 2 commits August 28, 2021 20:53
Add Upload Button to UploadAlt.tsx
@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 6 alerts when merging 1a58698 into 77f44b8 - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Aug 28, 2021

This pull request introduces 2 alerts when merging 3cde813 into 77f44b8 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

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.

4 participants