Skip to content

Fix problem showing emails from different participants #6503

Merged
ivan-mr merged 2 commits intodevelopfrom
fix/show_email_twice
Sep 18, 2020
Merged

Fix problem showing emails from different participants #6503
ivan-mr merged 2 commits intodevelopfrom
fix/show_email_twice

Conversation

@leio10
Copy link
Copy Markdown
Contributor

@leio10 leio10 commented Sep 7, 2020

🎩 What? Why?

When an admin try to view a participant email in the admin zone it works fine. But if the admin tries to see the email of another user without refreshing the page it keeps showing the same email.

This seems to be a jQuery problem. It works well after switching that code to vanilla JS.

📌 Related Issues

📋 Subtasks

  • Add tests

📷 Screenshots (optional)

Description

@leio10 leio10 changed the title OnDon't show the same email Fix problem showing emails from different participants Sep 7, 2020
@mrcasals
Copy link
Copy Markdown
Contributor

mrcasals commented Sep 9, 2020

@decidim/core can you check this out, please?

@mrcasals
Copy link
Copy Markdown
Contributor

@decidim/core this PR fixes a bug, can you check it out please? 😄

Copy link
Copy Markdown
Contributor

@ivan-mr ivan-mr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good job @leio10. Thanks!

@ivan-mr ivan-mr merged commit 06b4276 into develop Sep 18, 2020
@ivan-mr ivan-mr deleted the fix/show_email_twice branch September 18, 2020 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow to see a participant's email from the admin panel

3 participants