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

Fix(web-react): Removing unused prop #DS-933 #1106

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

pavelklibani
Copy link
Contributor

Description

  • unused prop isDisabled was removed from FileUploader component
  • isDisabled is still usable in FileUploaderInput component

Issue reference

web-react: FileUploader má prop isDisabled, co nic nedělá

@pavelklibani pavelklibani self-assigned this Oct 23, 2023
@pavelklibani pavelklibani requested review from literat and a team as code owners October 23, 2023 10:56
@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for spirit-design-system-react canceled.

Name Link
🔨 Latest commit 43634db
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-react/deploys/6537870cddf36000084e5a2a

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for spirit-design-system-storybook canceled.

Name Link
🔨 Latest commit 43634db
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-storybook/deploys/6537870d42b9c80008aec518

@github-actions github-actions bot added the bug Something isn't working label Oct 23, 2023
@nx-cloud
Copy link

nx-cloud bot commented Oct 23, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 43634db. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 3 targets

Sent with 💌 from NxCloud.

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for spirit-design-system-demo canceled.

Name Link
🔨 Latest commit 43634db
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-demo/deploys/6537870c85eeb20008bedda5

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for spirit-design-system-validations canceled.

Name Link
🔨 Latest commit 43634db
🔍 Latest deploy log https://app.netlify.com/sites/spirit-design-system-validations/deploys/6537870ca2754000087e2e4b

@pavelklibani pavelklibani force-pushed the fix/DS-933-fileuploader-isdisabled-type branch from 0a3cef6 to 650b1d1 Compare October 23, 2023 11:07
@coveralls
Copy link

Coverage Status

coverage: 70.405% (-26.1%) from 96.46% when pulling 650b1d1 on fix/DS-933-fileuploader-isdisabled-type into 0cb30d5 on main.

@literat
Copy link
Collaborator

literat commented Oct 23, 2023

Can you please change the commit message to

Fix(web-react): Remove unused `isDisabled` prop from FileUploader #DS-933

I will be more than happy if in the commit message will be the name of the component, which is affected by the change. Thanks :-)

@literat
Copy link
Collaborator

literat commented Oct 23, 2023

- unused prop `isDisabled` was removed from FileUploader
component #DS-933
@pavelklibani pavelklibani force-pushed the fix/DS-933-fileuploader-isdisabled-type branch from 650b1d1 to 43634db Compare October 24, 2023 08:57
@pavelklibani
Copy link
Contributor Author

Can you please change the commit message to

Fix(web-react): Remove unused `isDisabled` prop from FileUploader #DS-933

I will be more than happy if in the commit message will be the name of the component, which is affected by the change. Thanks :-)

Commit message changed 👍

@pavelklibani pavelklibani merged commit ab66984 into main Oct 24, 2023
19 checks passed
@pavelklibani pavelklibani deleted the fix/DS-933-fileuploader-isdisabled-type branch October 24, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants