MarinoDev/leptos_form_tool
Archived
5
0
Fork 0
generated from mitchell/rust_template
Code Issues 3 Pull Requests Packages Projects Releases Wiki Activity

Dont provide style and FD for getting validator #24

New Issue
Closed
opened 2024-06-11 13:29:33 +00:00 by mitchell · 4 comments
mitchell commented 2024-06-11 13:29:33 +00:00
Owner
Copy Link
Copy Source

Now you need to provide a style and form data even when you are just building the validator. When we add a context, you would also need to provide that.

This would be solved by reintroducing the Validation builder variant.

Now you need to provide a style and form data even when you are just building the validator. When we add a context, you would also need to provide that. This would be solved by reintroducing the Validation builder variant.
mitchell commented 2024-06-11 17:03:57 +00:00
Author
Owner
Copy Link
Copy Source

The context being required is the big problem, as it's going to be mostly render context stuff

The context being required is the big problem, as it's going to be mostly render context stuff
mitchell commented 2024-06-12 19:10:11 +00:00
Author
Owner
Copy Link
Copy Source

Since we are putting the Validation Builder back in, it might make sense to get rid of the RenderFn

Since we are putting the Validation Builder back in, it might make sense to get rid of the RenderFn
mitchell commented 2024-06-12 20:12:31 +00:00
Author
Owner
Copy Link
Copy Source

The problem is, if we pass the context in addition to the builder when building components (which is necessary for dynamic select options) then the context must be present for building the form and the validator.

The other option would be to pass an optional Context, which would just be None when getting the Validator.

The problem is, if we pass the context in addition to the builder when building components (which is necessary for dynamic select options) then the context must be present for building the form and the validator. The other option would be to pass an optional Context, which would just be None when getting the Validator.
mitchell commented 2024-06-12 20:19:02 +00:00
Author
Owner
Copy Link
Copy Source

It looks like we can even get rid of the fd field in the builder, since it's passed in in the render fn. Perhaps the same with the style.

It looks like we can even get rid of the `fd` field in the builder, since it's passed in in the render fn. Perhaps the same with the style.
This repo is archived. You cannot comment on issues.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Kind/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
diana drake (Drake Marino) gaven mitchell (Mitchell Marino) robert
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MarinoDev/leptos_form_tool#24