Skip to content

Fix error when exporting user data#6612

Merged
ivan-mr merged 13 commits intodecidim:developfrom
alecslupu-pfa:fix/user-exporter-job
Oct 13, 2020
Merged

Fix error when exporting user data#6612
ivan-mr merged 13 commits intodecidim:developfrom
alecslupu-pfa:fix/user-exporter-job

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

🎩 What? Why?

When attempting to export the user data ( http://localhost:3000/data_portability ) the asynchronous job that performs the data aggregation ( DataPortabilityExportJob ) fails with CarrierWave::IntegrityError (The file is not attached to any organization), due to fact that no model is bound to uploader file.

📌 Related Issues

Link your PR to an issue

  • Related to #?
  • Fixes #?

Testing

Describe the best way to test or validate your PR.

📋 Checklist

🚨 Please review the guidelines for contributing to this repository.

  • CONSIDER adding a unit test if your PR resolves an issue.
  • ✔️ DO check open PR's to avoid duplicates.
  • ✔️ DO keep pull requests small so they can be easily reviewed.
  • ✔️ DO build locally before pushing.
  • ✔️ DO make sure tests pass.
  • ✔️ DO make sure any new changes are documented in docs/.
  • ✔️ DO add and modify seeds if necessary.
  • ✔️ DO add CHANGELOG upgrade notes if required.
  • ✔️ DO add to GraphQL API if there are new public fields.
  • ✔️ DO add link to MetaDecidim if it's a new feature.
  • AVOID breaking the continuous integration build.
  • AVOID making significant changes to the overall architecture.

📷 Screenshots

Please add screenshots of the changes you're proposing
Description

♥️ Thank you!

@tramuntanal
Copy link
Copy Markdown
Contributor

tramuntanal commented Oct 7, 2020

#6609 has been merged, you can now rebase with develop @alecslupu

@alecslupu
Copy link
Copy Markdown
Contributor Author

@tramuntanal the rebase is done.

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.

Looks good to me. Thanks @alecslupu !

@ivan-mr ivan-mr self-assigned this Oct 13, 2020
@ivan-mr ivan-mr merged commit 17a264a into decidim:develop Oct 13, 2020
@alecslupu alecslupu deleted the fix/user-exporter-job branch January 26, 2021 19:22
mrcasals pushed a commit that referenced this pull request Feb 9, 2021
Co-authored-by: Alexandru Emil Lupu <contact@alecslupu.ro>
@mrcasals mrcasals added the type: fix PRs that implement a fix for a bug label Feb 25, 2021
@mrcasals mrcasals changed the title Exporting user data throws CarrierWave::IntegrityError (The file is not attached to any organization) Fix error when exporting user data Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in-review module: core type: fix PRs that implement a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants