Merge pull request #457 from rvsia/fixPropTypes

fix(all): fix proptypes accross all packages

Author: Hyperkid123

packages/common/src/form-template.js

@@ -63,9 +63,9 @@ export const FormControls = ({
 FormControls.propTypes = {
   onCancel: PropTypes.func,
   onReset: PropTypes.func,
-  submitLabel: PropTypes.string,
-  cancelLabel: PropTypes.string,
-  resetLabel: PropTypes.string,
+  submitLabel: PropTypes.node,
+  cancelLabel: PropTypes.node,
+  resetLabel: PropTypes.node,
   canReset: PropTypes.bool,
   disableSubmit: PropTypes.bool,
   buttonOrder: PropTypes.arrayOf(PropTypes.string),

packages/common/src/prop-types-templates.js

@@ -1,6 +1,6 @@
 import PropTypes from 'prop-types';
 
-export const optionsPropType = PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.string.isRequired, value: PropTypes.any }));
+export const optionsPropType = PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.node.isRequired, value: PropTypes.any }));
 
 export const meta = PropTypes.shape({
   active: PropTypes.bool,

packages/mui-component-mapper/src/files/plain-text.js

@@ -10,7 +10,7 @@ const PlainText = ({ label, name, component, ...props }) =>
   ));
 
 PlainText.propTypes = {
-  label: PropTypes.string.isRequired,
+  label: PropTypes.node.isRequired,
   name: PropTypes.string.isRequired,
   component: PropTypes.any
 };

packages/mui-component-mapper/src/files/radio.js

@@ -43,7 +43,7 @@ const RadioOption = ({ name, option, isDisabled, isReadOnly, FormControlLabelPro
 
 RadioOption.propTypes = {
   name: PropTypes.string.isRequired,
-  option: PropTypes.shape({ label: PropTypes.string.isRequired, value: PropTypes.any.isRequired }).isRequired,
+  option: PropTypes.shape({ label: PropTypes.node.isRequired, value: PropTypes.any.isRequired }).isRequired,
   isReadOnly: PropTypes.bool,
   isDisabled: PropTypes.bool,
   FormControlLabelProps: PropTypes.object,

packages/mui-component-mapper/src/files/sub-form.js

@@ -37,8 +37,8 @@ const SubForm = ({ fields, title, description, component, ...rest }) => {
 
 SubForm.propTypes = {
   fields: PropTypes.oneOfType([PropTypes.object, PropTypes.array]).isRequired,
-  title: PropTypes.string,
-  description: PropTypes.string,
+  title: PropTypes.node,
+  description: PropTypes.node,
   component: PropTypes.any
 };
 

packages/mui-component-mapper/src/files/wizard/wizard-step.js

@@ -13,8 +13,8 @@ const WizardStep = ({ title, description, fields, formOptions, ...rest }) => (
 );
 
 WizardStep.propTypes = {
-  title: PropTypes.string,
-  description: PropTypes.string,
+  title: PropTypes.node,
+  description: PropTypes.node,
   fields: PropTypes.array.isRequired,
   formOptions: PropTypes.shape({
     renderForm: PropTypes.func.isRequired

packages/pf3-component-mapper/src/common/form-wrapper.js

@@ -27,11 +27,11 @@ Pf3FormGroup.propTypes = {
   meta: PropTypes.shape({ error: PropTypes.string }).isRequired,
   validateOnMount: PropTypes.bool,
   hideLabel: PropTypes.bool,
-  label: PropTypes.string,
+  label: PropTypes.node,
   noCheckboxLabel: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   children: PropTypes.oneOfType([PropTypes.node, PropTypes.arrayOf(PropTypes.node)]).isRequired,
   inputAddon: PropTypes.shape({ fields: PropTypes.array })
 };

packages/pf3-component-mapper/src/files/button.js

@@ -9,7 +9,7 @@ export const Button = ({ label, variant, dataType, validate, children, component
 );
 
 Button.propTypes = {
-  label: PropTypes.string.isRequired,
+  label: PropTypes.node.isRequired,
   variant: PropTypes.string,
   className: PropTypes.string,
   dataType: PropTypes.any, // should be inside inner props or something

packages/pf3-component-mapper/src/files/checkbox.js

@@ -19,18 +19,18 @@ const SingleCheckbox = (props) => {
 };
 
 SingleCheckbox.propTypes = {
-  label: PropTypes.string,
+  label: PropTypes.node,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
+  helperText: PropTypes.node,
   isDisabled: PropTypes.bool,
-  description: PropTypes.string
+  description: PropTypes.node
 };
 
 const Checkbox = ({ options, ...props }) => (options ? <MultipleChoiceList options={options} {...props} /> : <SingleCheckbox {...props} />);
 
 Checkbox.propTypes = {
-  options: PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.string, value: PropTypes.any }))
+  options: PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.node, value: PropTypes.any }))
 };
 
 export default Checkbox;

packages/pf3-component-mapper/src/files/date-picker.js

@@ -26,12 +26,12 @@ const DatePicker = (props) => {
 DatePicker.propTypes = {
   meta: PropTypes.object,
   validateOnMount: PropTypes.bool,
-  label: PropTypes.string,
+  label: PropTypes.node,
   hideLabel: PropTypes.bool,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   input: PropTypes.object,
   placeholder: PropTypes.string,
   isDisabled: PropTypes.bool,

packages/pf3-component-mapper/src/files/plain-text.js

@@ -33,7 +33,7 @@ const PlainText = ({ variant, label, name }) =>
 
 PlainText.propTypes = {
   variant: PropTypes.oneOf(validTextFields),
-  label: PropTypes.string.isRequired,
+  label: PropTypes.node.isRequired,
   name: PropTypes.string.isRequired
 };
 

packages/pf3-component-mapper/src/files/radio.js

@@ -44,14 +44,14 @@ const Radio = ({ name, isRequired, isDisabled, isReadOnly, validateOnMount, help
 Radio.propTypes = {
   name: PropTypes.string.isRequired,
   validateOnMount: PropTypes.bool,
-  label: PropTypes.string,
+  label: PropTypes.node,
   hideLabel: PropTypes.bool,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   isDisabled: PropTypes.bool,
-  options: PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.string, value: PropTypes.any })),
+  options: PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.node, value: PropTypes.any })),
   inputAddon: PropTypes.shape({ fields: PropTypes.array })
 };
 

packages/pf3-component-mapper/src/files/select.js

@@ -29,12 +29,12 @@ const Select = (props) => {
 Select.propTypes = {
   meta: PropTypes.object,
   validateOnMount: PropTypes.bool,
-  label: PropTypes.string,
+  label: PropTypes.node,
   hideLabel: PropTypes.bool,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   input: PropTypes.object,
   placeholder: PropTypes.string,
   isDisabled: PropTypes.bool,

packages/pf3-component-mapper/src/files/select/select.js

@@ -80,7 +80,7 @@ const SelectTitle = ({ title, classNamePrefix, isClearable, value, onClear, isFe
 );
 
 SelectTitle.propTypes = {
-  title: PropTypes.string.isRequired,
+  title: PropTypes.node.isRequired,
   classNamePrefix: PropTypes.string,
   isClearable: PropTypes.bool,
   value: PropTypes.any,

packages/pf3-component-mapper/src/files/sub-form.js

@@ -15,8 +15,8 @@ const SubForm = ({ fields, title, description, component, ...rest }) => {
 
 SubForm.propTypes = {
   fields: PropTypes.oneOfType([PropTypes.object, PropTypes.array]).isRequired,
-  title: PropTypes.string,
-  description: PropTypes.string,
+  title: PropTypes.node,
+  description: PropTypes.node,
   component: PropTypes.any
 };
 

packages/pf3-component-mapper/src/files/switch.js

@@ -42,12 +42,12 @@ const Switch = (props) => {
 Switch.propTypes = {
   meta: PropTypes.object,
   validateOnMount: PropTypes.bool,
-  label: PropTypes.string,
+  label: PropTypes.node,
   hideLabel: PropTypes.bool,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   input: PropTypes.object,
   placeholder: PropTypes.string,
   isDisabled: PropTypes.bool,

packages/pf3-component-mapper/src/files/text-field.js

@@ -40,12 +40,12 @@ const TextField = (props) => {
 TextField.propTypes = {
   meta: PropTypes.object,
   validateOnMount: PropTypes.bool,
-  label: PropTypes.string,
+  label: PropTypes.node,
   hideLabel: PropTypes.bool,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   input: PropTypes.object,
   placeholder: PropTypes.string,
   isDisabled: PropTypes.bool,

packages/pf3-component-mapper/src/files/textarea.js

@@ -40,12 +40,12 @@ const Textarea = (props) => {
 Textarea.propTypes = {
   meta: PropTypes.object,
   validateOnMount: PropTypes.bool,
-  label: PropTypes.string,
+  label: PropTypes.node,
   hideLabel: PropTypes.bool,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   input: PropTypes.object,
   placeholder: PropTypes.string,
   isDisabled: PropTypes.bool,

packages/pf3-component-mapper/src/files/wizard.js

@@ -55,7 +55,7 @@ const WizardInternal = ({ title, buttonLabels, stepsInfo, inModal, onKeyDown, fo
 };
 
 WizardInternal.propTypes = {
-  title: PropTypes.string,
+  title: PropTypes.node,
   buttonLabels: PropTypes.object,
   stepsInfo: PropTypes.array,
   inModal: PropTypes.bool,

packages/pf3-component-mapper/src/form-fields/date-time-picker/date-time-picker.js

@@ -151,7 +151,7 @@ DateTimePicker.propTypes = {
   placeholder: PropTypes.string,
   variant: PropTypes.string,
   locale: PropTypes.string,
-  todayButtonLabel: PropTypes.string,
+  todayButtonLabel: PropTypes.node,
   showTodayButton: PropTypes.bool,
   isDisabled: PropTypes.bool,
   disabledDays: PropTypes.array,

packages/pf3-component-mapper/src/form-fields/date-time-picker/popover-root.js

@@ -96,7 +96,7 @@ PopoverRoot.propTypes = {
   variant: PropTypes.string,
   onHourChange: PropTypes.func.isRequired,
   onMinuteChange: PropTypes.func.isRequired,
-  todayButtonLabel: PropTypes.string,
+  todayButtonLabel: PropTypes.node,
   showTodayButton: PropTypes.bool,
   disabledDays: PropTypes.array
 };

packages/pf3-component-mapper/src/form-fields/required-label.js

@@ -8,7 +8,7 @@ const RequiredLabel = ({ label }) => (
 );
 
 RequiredLabel.propTypes = {
-  label: PropTypes.string.isRequired
+  label: PropTypes.node.isRequired
 };
 
 export default RequiredLabel;

packages/pf4-component-mapper/src/common/form-group.js

@@ -25,11 +25,11 @@ const FormGroup = ({ label, isRequired, helperText, meta, description, hideLabel
 };
 
 FormGroup.propTypes = {
-  label: PropTypes.string,
+  label: PropTypes.node,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
+  helperText: PropTypes.node,
   meta: PropTypes.object.isRequired,
-  description: PropTypes.string,
+  description: PropTypes.node,
   hideLabel: PropTypes.bool,
   id: PropTypes.string.isRequired,
   children: PropTypes.oneOfType([PropTypes.element, PropTypes.arrayOf(PropTypes.element)]).isRequired

packages/pf4-component-mapper/src/common/select/multi-value-container.js

@@ -10,7 +10,7 @@ const MultiValueContainer = (props) => (
 
 MultiValueContainer.propTypes = {
   children: PropTypes.oneOfType([PropTypes.node, PropTypes.arrayOf(PropTypes.node)]).isRequired,
-  data: PropTypes.shape({ label: PropTypes.string.isRequired }).isRequired,
+  data: PropTypes.shape({ label: PropTypes.node.isRequired }).isRequired,
   className: PropTypes.string
 };
 

packages/pf4-component-mapper/src/common/select/select.js

@@ -53,8 +53,8 @@ const Select = ({ selectVariant, menuIsPortal, ...props }) => {
 Select.propTypes = {
   selectVariant: PropTypes.oneOf(['default', 'createable']),
   isSearchable: PropTypes.bool,
-  showMoreLabel: PropTypes.string,
-  showLessLabel: PropTypes.string,
+  showMoreLabel: PropTypes.node,
+  showLessLabel: PropTypes.node,
   simpleValue: PropTypes.bool,
   value: PropTypes.any,
   options: PropTypes.arrayOf(

packages/pf4-component-mapper/src/common/select/value-container.js

@@ -37,8 +37,8 @@ ValueContainer.propTypes = {
   getStyles: PropTypes.func.isRequired,
   children: PropTypes.oneOfType([PropTypes.node, PropTypes.arrayOf(PropTypes.node)]).isRequired,
   selectProps: PropTypes.shape({
-    showLessLabel: PropTypes.string,
-    showMoreLabel: PropTypes.string
+    showLessLabel: PropTypes.node,
+    showMoreLabel: PropTypes.node
   })
 };
 

packages/pf4-component-mapper/src/files/checkbox.js

@@ -24,19 +24,19 @@ const SingleCheckbox = (props) => {
 };
 
 SingleCheckbox.propTypes = {
-  label: PropTypes.string,
+  label: PropTypes.node,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
   isDisabled: PropTypes.bool,
   id: PropTypes.string
 };
 
 const Checkbox = ({ options, ...props }) => (options ? <MultipleChoiceList options={options} {...props} /> : <SingleCheckbox {...props} />);
 
 Checkbox.propTypes = {
-  options: PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.string, value: PropTypes.any }))
+  options: PropTypes.arrayOf(PropTypes.shape({ label: PropTypes.node, value: PropTypes.any }))
 };
 
 export default Checkbox;

packages/pf4-component-mapper/src/files/date-picker.js

@@ -22,12 +22,12 @@ const DatePicker = (props) => {
 };
 
 DatePicker.propTypes = {
-  label: PropTypes.string,
+  label: PropTypes.node,
   isReadOnly: PropTypes.bool,
   isRequired: PropTypes.bool,
-  helperText: PropTypes.string,
-  description: PropTypes.string,
-  hideLabel: PropTypes.string,
+  helperText: PropTypes.node,
+  description: PropTypes.node,
+  hideLabel: PropTypes.bool,
   isDisabled: PropTypes.bool,
   id: PropTypes.string
 };