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.

Sign up for GitHub

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

Jump to bottom

Create interactive AssetContainer #35

Closed
1 task done
skorphil opened this issue Feb 19, 2024 · 2 comments · Fixed by #40
Closed
1 task done

Create interactive AssetContainer #35

skorphil opened this issue Feb 19, 2024 · 2 comments · Fixed by #40
Assignees
@skorphil
Labels
enhancement New feature or request

Comments

@skorphil
Copy link
Owner

skorphil commented Feb 19, 2024 •
edited
Loading

Based on Data flow concept - https://github.com/skorphil/nextjs-form/blob/main/docs/data-flow.md

  • FormData changing based on asset fields
  • FormData is changing based on deletion/addition of asset

implement handleAssetDelete method

  • Where to store handlers?
    Inside component? In componentFolder? In handlers folder?
  • For now stored in component, where the state(to be manipulated with handlers) is
@skorphil skorphil added the enhancement New feature or request label Feb 19, 2024
@skorphil skorphil self-assigned this Feb 19, 2024
@skorphil
Copy link
Owner Author

Remaining tasks for future enhancements:

@skorphil
Copy link
Owner Author

https://www.chromatic.com/component?appId=65b2882ac1586a6b21cba0dd&csfId=recordform-assetcontainer&buildNumber=39&k=65d47b595a7a61a7a3a80575-1200px-interactive-true&h=7&b=-1

@skorphil skorphil added this to the Dynamic form milestone Mar 11, 2024
@skorphil skorphil linked a pull request Mar 11, 2024 that will close this issue
added interactivity to AssetContainer #40
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skorphil skorphil

Labels
enhancement New feature or request
Projects
piggy-tracker-form
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

added interactivity to AssetContainer skorphil/nextjs-form
1 participant
@skorphil