Skip to content

Fixing error caused by Missing Organization (#6612)#7317

Merged
mrcasals merged 1 commit intodecidim:release/0.23-stablefrom
armandfardeau:backport/fix/user-exporter-job
Feb 9, 2021
Merged

Fixing error caused by Missing Organization (#6612)#7317
mrcasals merged 1 commit intodecidim:release/0.23-stablefrom
armandfardeau:backport/fix/user-exporter-job

Conversation

@armandfardeau
Copy link
Copy Markdown
Contributor

@armandfardeau armandfardeau commented Feb 8, 2021

🎩 What? Why?

Backport

On a new instance, we found a large number of errors. This Backport solves the issue

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.

#6612

📌 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!

@armandfardeau armandfardeau changed the title fixing error caused by Missing Organization (#6612) Fixing error caused by Missing Organization (#6612) Feb 8, 2021
@mrcasals
Copy link
Copy Markdown
Contributor

mrcasals commented Feb 9, 2021

Looks good to me, but I'm waiting until the CI job is ifnished just in case (I restarted the Admin one)

@mrcasals mrcasals merged commit 6b1d322 into decidim:release/0.23-stable Feb 9, 2021
@armandfardeau armandfardeau deleted the backport/fix/user-exporter-job branch February 9, 2021 09:23
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

Successfully merging this pull request may close these issues.

3 participants