Skip to content

Support Arrays as well as Strings.#50

Open
ignu wants to merge 1 commit into
mdeering:masterfrom
ignu:master
Open

Support Arrays as well as Strings.#50
ignu wants to merge 1 commit into
mdeering:masterfrom
ignu:master

Conversation

@ignu

@ignu ignu commented Jan 19, 2014

Copy link
Copy Markdown

I found that I needed this in order to use Array types with attribute_normalizer.

@dgilperez

Copy link
Copy Markdown

👍

@ignu

ignu commented Jan 20, 2014

Copy link
Copy Markdown
Author

Sorry about that, pushed a new commit.

@mdeering

Copy link
Copy Markdown
Owner

Great idea. Thanks.

@acurley

acurley commented Mar 28, 2014

Copy link
Copy Markdown

Any idea when this pull request will be complete?

@mdeering

Copy link
Copy Markdown
Owner

What is your use case for this? Seems like something you could do easily by adding your own normalizer in an initializer. Wondering if this is a common enough use case to warrant inclusion.

@nikolai-b

Copy link
Copy Markdown

It would be useful for me 👍

@mdeering mdeering added this to the 2.0 milestone Aug 15, 2015
@nikolai-b

Copy link
Copy Markdown

I realised I didn't give a good use case, sorry.

Postgres allows me to save an array, normally I want to strip and downcase items before going into the array but for postcodes I want to upcase them. I could make a custom normalizer for array strip downcase and one for array strip upcase but it would be much nicer to use the existing normalizers.

@AlexEscalante

Copy link
Copy Markdown

I also would like this 👍

@dmeremyanin

Copy link
Copy Markdown
Contributor

I've just added arrays normalization to normalizr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants