Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Initial FreeformInputWidget #48

Merged
merged 16 commits into from
Oct 5, 2023
Merged

Initial FreeformInputWidget #48

merged 16 commits into from
Oct 5, 2023

Conversation

pelusanchez
Copy link
Collaborator

No description provided.

@FussyDuck
Copy link

FussyDuck commented Sep 18, 2023

CLA assistant check
All committers have signed the CLA.

/**
* Permitted selections for the widget.
*/
constraints?: string[]
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can get rid of this for now if constraints are not going to be already applied to this widget.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Dropped constraints for the moment, also bypass and all dependencies of constraints.

@@ -0,0 +1,366 @@
import React from 'react'

import { FreeformInputWidget, TooltipProvider } from '../../src'
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we need to make sure this widget works also when it's inside an ExclusiveGroupWidget. See https://github.com/ecmwf-projects/cads-ui-library/blob/main/cypress/component/ExclusiveGroupWidget.cy.tsx for examples.

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added at 667e176 .

@pelusanchez pelusanchez changed the title DRAFT - Initial FreeformInputWidget Initial FreeformInputWidget Sep 25, 2023
</WidgetHeader>
<ReservedSpace data-stylizable='widget freeform-input reserved-error-space'>
{required && value?.length ? (
<Error>The field is required.</Error>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be "This field is required"

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, it seems that the message does not disappear even if the value is present.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed!

CHANGELOG.md Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

Please, update the CHANGELOG as well.

@valentinogagliardi
Copy link
Contributor

The initial version of FreeformInputWidget without validation looks good to me. You can release the major version.

@pelusanchez
Copy link
Collaborator Author

@pelusanchez pelusanchez merged commit cb01522 into main Oct 5, 2023
6 checks passed
@pelusanchez pelusanchez deleted the COPDS-1192 branch February 7, 2024 14:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants