biothings / discovery-app

The Data Discovery Engine project by the CD2H Data working-group
http://discovery.biothings.io
Apache License 2.0
4 stars 12 forks source link

Change URL for an existing namespace #241

Closed ljgarcia closed 4 months ago

ljgarcia commented 10 months ago

Describe the bug Change URL for an existing namespace does not seem to do anything.

To Reproduce Steps to reproduce the behavior:

  1. Go to a used dashboard
  2. Click on the update button for any registered namespace
  3. On the pop up window choose "Change URL"
  4. On the text box fill in the new link
  5. Click "Go"
  6. Nothing happens

Expected behavior A clear and concise description of what you expected to happen. On the last step, I would expect to see something similar to what happen on the playground "Visualize your own schema", i.e., a mini report on whether the schema link could be loaded and whether it looks good and ready to go. If it is all clear, I would expect the namespace to be updated to the new link.

Screenshots If applicable, add screenshots to help explain your problem.

Additional context Add any other context about the problem here.

ljgarcia commented 10 months ago

Using developer tools on the browser, this is what I see

 SweetAlert2: "allowOutsideClick" parameter requires `backdrop` parameter to be set to `true`
c @ _nuxt/entry.971e7970.js:1559
Sb @ _nuxt/entry.971e7970.js:1617
_main @ _nuxt/entry.971e7970.js:1620
oo @ _nuxt/entry.971e7970.js:1620
Im @ _nuxt/entry.971e7970.js:1617
(anonymous) @ _nuxt/index.b91b0c88.js:1
Promise.then (async)
then @ _nuxt/entry.971e7970.js:1620
updateSelected @ _nuxt/index.b91b0c88.js:1
onClick @ _nuxt/index.b91b0c88.js:1
Ac @ _nuxt/entry.971e7970.js:1
Ga @ _nuxt/entry.971e7970.js:1
i @ _nuxt/entry.971e7970.js:1
_nuxt/index.b91b0c88.js:1 Uncaught (in promise) ReferenceError: data is not defined
    at _nuxt/index.b91b0c88.js:1:8271
marcodarko commented 10 months ago

@ljgarcia Hi, yes some libraries were updated during our migration and I think I missed that, I will try to push an update ASAP since it's an easy fix. Also working on all the other issues just FYI. Thank you!