Skip to content

Conversation

mgu
Copy link
Contributor

@mgu mgu commented Feb 24, 2017

catch up API changes in formidable-ui

@mgu mgu force-pushed the fix_help_text_in_api branch from 8ea7efc to 8e267c4 Compare February 24, 2017 12:47
@mgu mgu requested review from brunobord and wo0dyn February 24, 2017 12:48
Copy link
Contributor

@brunobord brunobord left a comment

Choose a reason for hiding this comment

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

could you please amend the Changelog?

Copy link
Contributor

@wo0dyn wo0dyn left a comment

Choose a reason for hiding this comment

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

I filled an issue to rename help_text to description in models (issue #190).

LGTM. Thanks for the contribution.

@wo0dyn wo0dyn dismissed brunobord’s stale review February 24, 2017 15:25

ChangeLog amended in 87387e6

@mgu mgu force-pushed the fix_help_text_in_api branch from 9e5ab0e to fe941c3 Compare March 1, 2017 14:55
@mgu mgu requested review from brunobord and wo0dyn March 1, 2017 15:04
@mgu
Copy link
Contributor Author

mgu commented Mar 1, 2017

New attempt.
Add possibility to load old JSON schema (using help_text instead of description)

Mickaël Guérin added 3 commits March 1, 2017 21:29
if 'help_text' attribute exists, then use this value for 'description'
@mgu mgu force-pushed the fix_help_text_in_api branch from fe941c3 to c5de7e9 Compare March 1, 2017 20:30
Copy link
Contributor

@wo0dyn wo0dyn left a comment

Choose a reason for hiding this comment

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

LGTM.

# XXX FIX ME: temporary fix
if 'help_text' in data:
data['description'] = data.pop('help_text')
return super(ItemSerializer, self).to_internal_value(data)
Copy link
Contributor

Choose a reason for hiding this comment

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

Here is the magic 😉

@mgu mgu merged commit 5f64eff into master Mar 6, 2017
@mgu mgu deleted the fix_help_text_in_api branch March 6, 2017 15:16
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