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

[#2189] Move to IATI statuses #2207

Merged
merged 35 commits into from
Jun 1, 2016
Merged

[#2189] Move to IATI statuses #2207

merged 35 commits into from
Jun 1, 2016

Conversation

damienallen
Copy link
Contributor

Fixes #2189

damienallen and others added 28 commits May 25, 2016 10:20
@akvobot
Copy link

akvobot commented May 26, 2016

Checklist

  • Test plan
  • Copyright header
  • Code formatting
  • Documentation

self.add_log('activity-status@code', 'status', 'invalid status code')
status = Project.STATUS_NONE
status = 0
Copy link
Contributor

Choose a reason for hiding this comment

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

This should probably be a string instead of integer, right?

@KasperBrandt
Copy link
Contributor

Also change the field in the 'old' project admin to use iati_status instead of status.

@damienallen
Copy link
Contributor Author

Only merge after #2206 has been merged

@damienallen damienallen changed the title #2189 Transition to IATI statuses (w/legacy support) [#2189] Transition to IATI statuses May 27, 2016
@damienallen damienallen changed the title [#2189] Transition to IATI statuses [#2189] Move to IATI statuses May 31, 2016
@KasperBrandt KasperBrandt merged commit 49bcf93 into develop Jun 1, 2016
@KasperBrandt KasperBrandt deleted the #2189-iati-status-field branch June 1, 2016 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants