feat: tags and postal code attributes

[franpb14]

Postal code is related to users and tags is related to members (for those who belong to more than one organisation, they can have different tags depending on the organisation).

Closes #623

Issue623.mp4

[sseerrggii]

😍

Just to be sure @franpb14 tags are shared only on the same organization? This is how it works in Posts

[franpb14]

Yes, tags is an attribute of the member model. So, for example, a user can have "tag1 and tag2" in one organisation and "otherTag and ultimaTag" in another. And if you want to see the tags of an organisation, just take the members of the organisation and extract the tags.

[franpb14]

I'm working on second step and I have seen a bug, with this PR it is not possible to search for publications by tags. I am looking into the reason for the failure but I wanted to warn you so that it doesn't merge. Sorry for the inconvenience.

[franpb14]

The problem was solved by indicating the table where to search for the tag in the query that was failing. Here is the commit that solves it.

[markets]

hi @franpb14 👋🏼

I think we can use this same branch and PR to add the other two points. The searcher and the autocomplete, seem quite "small" tasks (we have examples of both in current code base) and very tied to this code.

PS comment for latest commit 32f4ebe: this is a concern (a mixin with steroids) to be injected in other classes, so hardcoding the a table name seems wrong to me, it should use the table of the injected AR class, we can do it by using the table_name method, something like: where("? = ANY (#{table_name}.tags)", tag)

[franpb14]

All steps in #623 are complete. I have two videos, the first one is related to the searchers working on tags:

Issue_623_searcher.mp4

And the last video is related to tag autocompletion taking into account the members tags and how to show the tags in "/tags/alpha_grouped_index":

Issue_623_autocomplete.mp4

[sseerrggii]

OMG I love this feature, also I like really much that you added it on tags list page 👏 👏 👏

[markets]

hello @franpb14 very nice improvements 👏🏼 👏🏼 I just completed the review and added some comments. Let me know what do you think?

[franpb14]

Thank you very much @markets :). All changes are fine, I will make them between today and tomorrow.

[franpb14]

@markets all the feedback is incorporated and I have also added member in case of failure of the forms.

[markets]

@franpb14 pushed some refactoring 4d9d6a7

I'll take a second look tomorrow and we'll be ready for staging!

[markets]

@franpb14 @sseerrggii ready for staging! ✅ (i18n keys already added in Localeapp and pulled here)

[sseerrggii]

Tested manually ✔️