[Bug]: `Key Type` in create form is misleading; proposal to use `Data Type` about redisinsight HOT 7 CLOSED

Hey @ollopaci,
Thank you for checking the issue. I was not really sure if any of the issues I raised would be accepted.

Now that this one is accepted, wanted to know if it is open for me / anybody to contribute. Could not find any labels to know; what indicates any of the issue are open to contribution?

from redisinsight.

Hi @prnam. Of course it is open for anyone to contribute. We actually would really like to encourage that. Taking your proposal and will create a corresponding label for this. Thank you!

from redisinsight.

Thank @prnam for the observation. We will address this in the next release.

from redisinsight.

Thank you for your timely response @ollopaci . I have raised the PR that will close this issue. Looking forward for CI approval for first time contributors, review and hoping everything will be good to merge 🤞

from redisinsight.

Hi @prnam, we have discussed this issue internally one more time and came to the conclusion that the "Key Type" is also an acceptable term and should clearly define its purpose, also using the "Key type" will allow us to be consistent with RediSearch team.
But I'd really like to thank you a lot for your input and contribution, this is very helpful.

from redisinsight.

Okay, @ViktorStarostenko . Thank you for reviewing and sharing the decision. 🙂

I assume no action item from my end as such; you or the team will take necessary action on the issue and PR, right?

from redisinsight.

@prnam, yes, we will take care of them.

from redisinsight.