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

WYSIWYG "Link a DNN file" opens ADAM menu instead on first interaction if files are already in ADAM #3448

Open
6TELOIV opened this issue Aug 9, 2024 · 5 comments

Comments

@6TELOIV
Copy link
Contributor

6TELOIV commented Aug 9, 2024

I'm submitting a

[x] bug report => search github for a similar issue before submitting

...about

[x] edit experience / UI

Current Behavior / Expected Behavior

In a WYSIWYG field, if the user has uploaded a file with ADAM, then the second file upload button will open the ADAM UI instead of the DNN UI. Clicking the button 2 more times (to dismiss and bring back the menu) brings up the correct menu
image

Instructions to Reproduce the Problem

  1. Upload a file using ADAM in a WYSIWYG field
  2. Refresh the page
  3. Try to upload a file with DNN in that same WYSIWYG field
  4. Observe behavior described above

Your environment

  • 2sxc version(s): 17.09.00
  • Browser: all
  • DNN: 9.13.2
  • Hosting platform: any
  • Language: english
@6TELOIV
Copy link
Contributor Author

6TELOIV commented Sep 17, 2024

Based on the nature of this bug, I feel like the cause is this adam cache code that runs on first load.

From what I can tell, if there are no files in adam, it will return undefined and fall through to the "getAll" case. Otherwise, it'll return the adam files and set the adamItems to that (even if the adamConfig has usePortalRoot set, it still uses the wrong cache value)

I haven't been able to test that this is the cause, but it seems like the solution is to wrap the cache code on line 294 of adam-browser.component.ts in a check that usePortalRoot is false.

@6TELOIV
Copy link
Contributor Author

6TELOIV commented Oct 11, 2024

@iJungleboy have you or another contributor had a chance to look at this yet?

I didn't specify in the original post, but as a consequence of this bug, our editors are running into confusing 403 errors, as we let them upload files into a specific folderin DNN, but not ADAM, because their documents need to be organized and shared in multiple locations. I've tried to explain to them the workaround, but it would be more ideal if they could get to the correct menu on the first click instead of the third 😅

@iJungleboy
Copy link
Contributor

@6TELOIV would you like to sponsor 2h to get this prioritized?

@6TELOIV
Copy link
Contributor Author

6TELOIV commented Oct 14, 2024

Good point ;)

Donations/payments/contracts are a bit above my paygrade at my org; if you could estimate how much it would cost them, I can let them know.

@iJungleboy
Copy link
Contributor

Check out https://2sxc.org/en/contact and https://2sxc.org/en/contact/paypal

2 hours is my estimate for this.

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

No branches or pull requests

2 participants