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.

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

feat(symfony): describe MapUploadedFile property #2418

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jankal
Copy link

@jankal jankal commented Jan 7, 2025

Description

Added support for the #[MapUploadedFile] Symfony controller argument attribute.

What type of PR is this? (check all applicable)

  • Bug Fix
  • Feature
  • Refactor
  • Deprecation
  • Breaking Change
  • Documentation Update
  • CI

Checklist

  • I have made corresponding changes to the documentation (docs/)
  • I have made corresponding changes to the changelog (CHANGELOG.md)

@jankal jankal force-pushed the feature/map-uploaded-file branch 3 times, most recently from 5cc5688 to 9f80b3c Compare January 7, 2025 22:51
@jankal
Copy link
Author

jankal commented Jan 7, 2025

Had to update my branch a few times to get rid of a merge requirement - didn't expect master to have moved forward in between already.

@jankal jankal force-pushed the feature/map-uploaded-file branch from 9f80b3c to f41a273 Compare January 7, 2025 22:56
@jankal
Copy link
Author

jankal commented Jan 7, 2025

Do we need to update the major version in

major_version: 6
to make the docs linter pass?

@jankal jankal force-pushed the feature/map-uploaded-file branch from f41a273 to 45f5e86 Compare January 7, 2025 23:01
@jankal
Copy link
Author

jankal commented Jan 8, 2025

PHPStan fails as Symfony packages are locked to 7.0 by

When / how do we change this?

@DjordyKoert
Copy link
Collaborator

PHPStan fails as Symfony packages are locked to 7.0 by

When / how do we change this?

Fixed in #2419

@DjordyKoert
Copy link
Collaborator

Do we need to update the major version in

major_version: 6

to make the docs linter pass?

Yeah bumping that version and removing the old referenced to 6.x in the docs will fix it

@jankal jankal force-pushed the feature/map-uploaded-file branch from 45f5e86 to fab548d Compare January 8, 2025 12:18
@jankal
Copy link
Author

jankal commented Jan 8, 2025

I just rebased my branch on current master to make checks to include your changes from #2419.

As I do not fully understand the ReStructured docs versionadded tags, I will leave this upgrade from 6 to 7 to you @DjordyKoert

Copy link

codecov bot commented Jan 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.60%. Comparing base (13408ab) to head (fab548d).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2418      +/-   ##
==========================================
+ Coverage   89.53%   89.60%   +0.06%     
==========================================
  Files          78       79       +1     
  Lines        2915     2933      +18     
==========================================
+ Hits         2610     2628      +18     
  Misses        305      305              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants