feat(symfony): describe MapUploadedFile property

[jankal]

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]

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]

Do we need to update the major version in

NelmioApiDocBundle/.doctor-rst.yaml

Line 56 in 8e56941

major_version: 6

to make the docs linter pass?

[jankal]

PHPStan fails as Symfony packages are locked to 7.0 by

NelmioApiDocBundle/.github/workflows/continuous-integration.yml

Line 176 in 8e56941

symfony-version: "7.0.*"

When / how do we change this?

[DjordyKoert]

PHPStan fails as Symfony packages are locked to 7.0 by

NelmioApiDocBundle/.github/workflows/continuous-integration.yml

Line 176 in 8e56941

symfony-version: "7.0.*"

When / how do we change this?

Fixed in #2419

[DjordyKoert]

Do we need to update the major version in

NelmioApiDocBundle/.doctor-rst.yaml

Line 56 in 8e56941

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]

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

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.24%. Comparing base (8b4803f) to head (98f4627).
Report is 8 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2418      +/-   ##
==========================================
+ Coverage   90.18%   90.24%   +0.05%     
==========================================
  Files          94       95       +1     
  Lines        3067     3085      +18     
==========================================
+ Hits         2766     2784      +18     
  Misses        301      301              

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

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[DjordyKoert]

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

@jankal I do not have permission to edit your PR :)

[DjordyKoert]

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

@jankal I do not have permission to edit your PR :)

A merge/rebase with master should fix it

[jankal]

@DjordyKoert thx! So you just removed the references to 6.* from the docs?

That's what I didn't want to do because the info would be lost after my PR was merged and docs updated. Can you explain how users of the library will be able to figure out this info in the future? - I'm just curious :)

I did the rebase. This should be ready to merge then.

[jankal]

(The doctor seems to still have eissues... now bc of your change to require 4.*)

[DjordyKoert]

Thank you ❤️