fix #13453 Documentation Plugin doesn't generate Proper Type for the …

[meganoob1337]

fixes #13453

[markkaylor]

Thanks for this PR, I might be able to review it before the end of today, otherwise it will have to be early next week.

[meganoob1337]

Thanks for this PR, I might be able to review it before the end of today, otherwise it will have to be early next week.

If you have any questions im in the strapi discord aswell.

I hope the code is okay like this, its my first PR here and was just my Fix for it, so bear with me please :D

[markkaylor]

Thanks for submitting this PR, nice work! My comments are mostly about cleaning things up and code style. Let me know if you have any questions or if something isn't clear.

[meganoob1337]

@markkaylor thx for the review and sorry for all the small things. ill probably come to it next week, as im away for the next days

[meganoob1337]

Sorry, had corona, will start soon :)

[meganoob1337]

@markkaylor any updates here?

[markkaylor]

Sorry @meganoob1337 didn't see the notification that this PR was up for review again. I will take a look this week and get back to you.

[codecov]

Codecov Report

Merging #13458 (e68d494) into main (ce6c3f9) will increase coverage by 2.81%.
The diff coverage is n/a.

❗ Current head e68d494 differs from pull request most recent head 2143c2d. Consider uploading reports for the commit 2143c2d to get more accurate results

@@            Coverage Diff             @@
##             main   #13458      +/-   ##
==========================================
+ Coverage   55.12%   57.94%   +2.81%     
==========================================
  Files        1251     1006     -245     
  Lines       31493    22405    -9088     
  Branches     5688     3683    -2005     
==========================================
- Hits        17362    12982    -4380     
+ Misses      12316     8433    -3883     
+ Partials     1815      990     -825     

Flag	Coverage Δ
front	57.94% <ø> (ø)
unit	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
packages/core/utils/lib/sanitize/visitors/index.js
.../core/admin/server/validation/common-validators.js
packages/core/utils/lib/pipe-async.js
packages/core/upload/server/services/file.js
packages/core/admin/server/utils/index.js
packages/core/utils/lib/relations.js
...ckages/core/content-manager/server/services/uid.js
packages/core/utils/lib/traverse-entity.js
...ckages/core/database/lib/query/helpers/populate.js
...8n/server/migrations/content-type/disable/index.js
... and 235 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[meganoob1337]

@markkaylor i added your requested Changes, have a look please :) (had some vacation - took a bit longer )