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

Add UploadCaster for Tempest\Http\Upload class #723

Open
yassiNebeL opened this issue Nov 13, 2024 · 3 comments
Open

Add UploadCaster for Tempest\Http\Upload class #723

yassiNebeL opened this issue Nov 13, 2024 · 3 comments
Labels
Database Feature Request Request a feature that is not currently in the framework.

Comments

@yassiNebeL
Copy link
Contributor

Description

Just like with the Id and IdCaster.

Benefits

It ensures consistency and type safety, just like the Id class and IdCaster.
@yassiNebeL yassiNebeL added Feature Request Request a feature that is not currently in the framework. Triage New issues that need to be reviewed by the Tempest team. labels Nov 13, 2024
@yassiNebeL yassiNebeL changed the title Add ˋUploadCaster for ˋTempest\Http\Upload class Add ˋUploadCasterˋ for ˋTempest\Http\Upload` class Nov 13, 2024
@yassiNebeL
Copy link
Contributor Author

Let’s discuss how to handle multiple file uploads.

@yassiNebeL yassiNebeL changed the title Add ˋUploadCasterˋ for ˋTempest\Http\Upload` class Add UploadCaster for Tempest\Http\Upload class Nov 13, 2024
@innocenzi innocenzi added HTTP and removed Triage New issues that need to be reviewed by the Tempest team. labels Nov 13, 2024
@innocenzi innocenzi added Database and removed HTTP labels Nov 25, 2024
@yassiNebeL yassiNebeL mentioned this issue Nov 25, 2024
Closed
@yassiNebeL
Copy link
Contributor Author

Let’s discuss how to handle multiple file uploads.

Hello @brendt, please let me know if I can close this issue?

Thanks!

@innocenzi innocenzi changed the title Add UploadCaster for Tempest\Http\Upload class Add UploadCaster for Tempest\Http\Upload class Dec 6, 2024
@innocenzi innocenzi mentioned this issue Dec 6, 2024
Closed
@innocenzi
Copy link
Member

Let's keep this issue open, I closed the other one as duplicate.

@yassiNebeL you can re-open #775 if you still want to work on this. Please don't open other PRs or issues for the same topic, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Database Feature Request Request a feature that is not currently in the framework.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(database): add UploadCaster class yassiNebeL/tempest-framework
2 participants
@innocenzi @yassiNebeL