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

[Bug]: Unable to delete Legacy UA connection #5867

Closed
2 tasks done
cklenk opened this issue Nov 10, 2023 · 4 comments
Closed
2 tasks done

[Bug]: Unable to delete Legacy UA connection #5867

cklenk opened this issue Nov 10, 2023 · 4 comments

Comments

@cklenk
Copy link

cklenk commented Nov 10, 2023

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

Upgrade DNN site to 9.13.0
Went to connections
Tried to delete Legacy UA connection
Does not delete connection
Console error
image
image

Steps to reproduce?

  1. Go to settings
  2. Go to connections
  3. Try to delete existing Legacy UA connection

Current Behavior

Unable to delete connection

Expected Behavior

Delete legacy UA connection

Relevant log output

jquery.js?cdv=595:2  Uncaught Error: Syntax error, unrecognized expression: #connector-GoogleAnalytics(LegacyUA)
    at I.error (jquery.js?cdv=595:2:11769)
    at Y (jquery.js?cdv=595:2:17234)
    at re (jquery.js?cdv=595:2:20008)
    at Function.I [as find] (jquery.js?cdv=595:2:7520)
    at ce.fn.init.find (jquery.js?cdv=595:2:21991)
    at new ce.fn.init (jquery.js?cdv=595:2:22483)
    at ce (jquery.js?cdv=595:2:1077)
    at Connectors.js?cdv=595:181:45

Anything else?

No response

Affected Versions

9.13.0 (latest release)

What browsers are you seeing the problem on?

Microsoft Edge

Code of Conduct

  • I agree to follow this project's Code of Conduct
@valadas valadas added this to the Future: Patch milestone Nov 10, 2023
@valadas
Copy link
Contributor

valadas commented Nov 10, 2023

Good catch, the element ID is probably driven by the module name and parenthesis can't be used for an element id.

@X3Technology
Copy link
Contributor

Also cannot update an existing Connector. I cant change the value for Google Tag Manager

@david-poindexter
Copy link
Contributor

Ah - great catch! I'll see if I can look into this during tomorrow's co-coding session on Discord if nobody else has addressed it by then.

@david-poindexter
Copy link
Contributor

Closing per #5972

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants