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.

Sign up for GitHub

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

Jump to bottom

Custom import of IATI results #2442

Closed
zzgvh opened this issue Nov 15, 2016 · 1 comment
Closed

Custom import of IATI results #2442

zzgvh opened this issue Nov 15, 2016 · 1 comment
Assignees
@zzgvh

Comments

@zzgvh
Copy link
Contributor

zzgvh commented Nov 15, 2016

The EUTF want to be able to import Result data only using the IATI import. The regular import mappers will delete most content of a project if only <result> tags are in the XML.

To solve this a custom mapper is needed along with some modifications to IatiImport.model_mappers()
zzgvh added a commit that referenced this issue Nov 15, 2016
@zzgvh
[#2442] Create IATI import mapper that only imports certain objects
0df509d 
Modify IatiImport.model_mappers so that a mapper with the suffix "_only" only uses the Classes listed in its __all__ list.

Create Result_only mapper that only imports Result and children thereof.
zzgvh added a commit that referenced this issue Nov 15, 2016
@zzgvh
[#2442] Fix IATI import when dates are missing
b3d0b0f 
Fix Dates import mapper so it doesn't throw and error if there is no activity-date tag.
@zzgvh zzgvh self-assigned this Nov 15, 2016
zzgvh added a commit that referenced this issue Nov 15, 2016
@zzgvh
[#2442] Add test for Result_only IATI import mapper
323556c
zzgvh added a commit that referenced this issue Nov 15, 2016
@zzgvh
Merge pull request #2443 from akvo/#2442_only_mapper
21a23e0 
#2442 only mapper
Merging this myself 😛 Need to look into the coverage problem later
punchagan added a commit that referenced this issue Nov 16, 2016
@punchagan
[#2442] Minor test change to bring coverage to 100%
c662d92
zzgvh added a commit that referenced this issue Nov 16, 2016
@zzgvh
Merge pull request #2444 from akvo/#2442-fix-coverage
c35391c 
[#2442] Minor test change to bring coverage to 100%
Nice!
@zzgvh zzgvh mentioned this issue Nov 16, 2016
Merged
zzgvh added a commit that referenced this issue Nov 16, 2016
@zzgvh
[#2442] Add migration for the mapper
260be44 
The code works without it, but it's needed to make the migrations machinery happy in the long run.
@punchagan
Copy link
Contributor

Closed with #2445

punchagan added a commit that referenced this issue Nov 30, 2016
@punchagan
Merge pull request #2464 from akvo/#2442_missing_migration
0b65300 
[#2442] Add migration for the mapper
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zzgvh zzgvh

Labels
None yet
Projects
RSR
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zzgvh @punchagan