Sp-dev-fx-controls-react: Org Assets Issue in FilePicker - Your organization tab is not shown

Created on 1 Jul 2020  路  4Comments  路  Source: pnp/sp-dev-fx-controls-react

Category

[ ] Enhancement

[x ] Bug

[ ] Question

Version

1.19.0

Expected / Desired Behavior / Question

Your organisation tab shown to get images/files provided by your organization, where filepicker has not provided props to hide

Observed Behavior

Your organization is not displayed and when you debug there are issues in the code which makes the org tab not visible

Steps to Reproduce

  • Your org assets are in the root site collection
  • Add FilePicker controls
  • No Your organization

Further if this helps:

The issues are in the files

FilePicker.tsx

Line 58
The condition should be (instead of && operation || should be used and sometimes we are not passing this prop and will be undefined)
if (this.props.hideOrganisationalAssetTab != undefined || !this.props.hideOrganisationalAssetTab ) {

FileBrowserService.ts

Line 268
Function buildAbsoluteUrl , need to have "/" if Library url does not start with a "/"

OrgAssetsService.ts

Line 76
Null check for function _parseOrgAssetsLibraryItem 's
libItem.ThumbnailUrl

It will be best to do null check for any property of the object, not only the item we need.

Hope that helps.
I can also take it up, if my approach of the fix sounds reasonable.

Thanks,
Rabia

Thanks!

fixed-next-drop bug

All 4 comments

Thank you for reporting this issue. We will be triaging your incoming issue as soon as possible.

Any news regarding this issue?

Hi @rabwill!
Thanks for reporting that and providing steps for fix!

The fix will be available in the next release.
In a meanwhile you can test it in beta version.

BTW, I've added you Rabia as a contributor as you provided the solution! Thank you!

Wohoo!! Thanks so much @AJIXuMuK ! You are awesome 馃ぉ

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Arknev picture Arknev  路  3Comments

repolife picture repolife  路  7Comments

enti333 picture enti333  路  7Comments

lenain71 picture lenain71  路  3Comments

estruyf picture estruyf  路  6Comments