2

[FEATURE] move redis key creation to a common module · Issue #917 · juspay/hyper...

 1 year ago
source link: https://github.com/juspay/hyperswitch/issues/917
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

[FEATURE] move redis key creation to a common module #917

2 tasks done

Narayanbhat166 opened this issue Apr 19, 2023 · 13 comments

Comments

Feature Description

Few entities currently are stored in redis. We create a key for each entity. This key will be usually of the form {feature}_{merchant_id}_{extras}. The creation of this key is being done in multiple places. It will be better to move all of these functions to a common module, this will add uniformity and help someone to easily know the key format for a particular entity.

Possible Implementation

There can be a module that exports functions for each of the key creation. This function takes the dynamic variables ( like merchant_id and extra variables and build the key. The format of this function would be

fn create_access_token_key(arg1: String, arg2: String, ...) -> String {}

One may also think about using enums for this.

Have you spent some time to check if this feature request has been raised before?

  • I checked and didn't find similar issue

Have you read the Contributing Guidelines?

Are you willing to submit a PR?

No, but I'm happy to collaborate on a PR with someone else

Narayanbhat166

added C-feature Category: Feature request or enhancement good first issue Good for newcomers help wanted Extra attention is needed

labels

Apr 19, 2023

🙋🏻‍♂️

I'd like to draft a solution for this issue for you smile. I suspect based on the criteria that there could be quite a bit of workshopping involved to iron this out, but I thrive on feedback and would enjoy that greatly.

white_check_mark I have read the contribution guidelines
eyes I saw this item linked in TWIR 491

Hey @IsaacCloos, I've assigned this issue to you.

Member

Author

Thanks @IsaacCloos for showing interest. Yes that would be great. We can discuss about the implementation.

Contributor

I am interested in collaborating with someone on their project so I can learn the language while they get a sidekick.
Let me Know if this is something that could work for you.
I Would need you to explain and guide me through my first steps in the language (I already started learning) and set the standard for me in code reviews but after a short while I'm sure I'll become an asset to the project.
(I have some experience in Typescript and Java and have learned C a few years ago).

I'm sorry for the delay! I have become busy with work and juggling a few other issues. Please feel free to reassign this issue as you see fit since I noticed it hasn't been labeled as low priority. I don't want to hold up any work by squatting on it.

I see @EliKalter is showing interest 👀


I'd be happy to share my experience with setting up the project for local development and testing if somebody else gets stuck slightly_smiling_face

Contributor

Thanks @IsaacCloos , I appreciate the response, Honestly I'm not sure it was a good idea for me to offer help because I've never contributed to an open source before (so I'd need help with the most basic of tasks in that regard) nor do I know rust even remotely well (so I'd need help on that from as well), I'm thinking maybe I should learn rust better by myself and start contributing on other beginner friendly projects first and get back to contributing to rust projects some day down the line, what are your thoughts?
(I don't like wasting peoples time...)

SanchithHegde

added E-easy Effort: Should be easy to implement and would make a good first PR P-low Priority: Low C-refactor Category: Refactor

and removed C-feature Category: Feature request or enhancement

labels

May 1, 2023

Member

I'm sorry for the delay! I have become busy with work and juggling a few other issues.

@IsaacCloos No problem, we understand that contributors have a personal life, and contribute to our project during their free time and out of their own interest, not due to obligation.

Please feel free to reassign this issue as you see fit since I noticed it hasn't been labeled as low priority. I don't want to hold up any work by squatting on it.

I'm sorry for the confusion, this is indeed a low priority refactor and does not block any of our work. It's a good-to-have change, and not an immediate necessity. I've updated the labels on this issue to reflect the same.

Let us know if you're still interested in taking a look at this, we'll help @EliKalter with any of the other issues they've expressed their interest on.

Member

[...] Honestly I'm not sure it was a good idea for me to offer help because I've never contributed to an open source before (so I'd need help with the most basic of tasks in that regard) nor do I know rust even remotely well (so I'd need help on that from as well), I'm thinking maybe I should learn rust better by myself and start contributing on other beginner friendly projects first and get back to contributing to rust projects some day down the line, what are your thoughts? (I don't like wasting peoples time...)

Folks being new to Rust and/or open source contributions is completely fine with us. We all started somewhere. Among the issues you expressed your interest on, #899 seems to be a pretty simple change. I can assign it to you and you can work on it at your own pace. Would that work for you? Feel free to ask any more questions you may have on that thread.

Contributor

[...] Honestly I'm not sure it was a good idea for me to offer help because I've never contributed to an open source before (so I'd need help with the most basic of tasks in that regard) nor do I know rust even remotely well (so I'd need help on that from as well), I'm thinking maybe I should learn rust better by myself and start contributing on other beginner friendly projects first and get back to contributing to rust projects some day down the line, what are your thoughts? (I don't like wasting peoples time...)

Folks being new to Rust and/or open source contributions is completely fine with us. We all started somewhere. Among the issues you expressed your interest on, #899 seems to be a pretty simple change. I can assign it to you and you can work on it at your own pace. Would that work for you? Feel free to ask any more questions you may have on that thread.

Thanks @SanchithHegde , I assume this thread is being taken care of by @IsaacCloos and that's way you suggested #899?!
I'll take a look at #899 and hopefully feel comfortable taking care of it, anyway I'll move my comments to that thread now, thank you

Member

I assume this thread is being taken care of by @IsaacCloos and that's way you suggested #899?!

I suggested #899 since we're not yet sure if @IsaacCloos would take this one up.

IsaacCloos

removed their assignment

May 9, 2023

SanchithHegde

added S-unassigned Status: This issue has no one assigned to address it

and removed S-unassigned Status: This issue has no one assigned to address it

labels

May 10, 2023

@EliKalter I'll be assigning this to you, as discussed over Discord.

EliKalter reacted with thumbs up emoji

@EliKalter Please let us know if you're still working on this issue or still interested in working on it. There hasn't been any recent activity on this thread, or any open PRs.

We hope to hear from you by the end of June 2023, or we'll open this for other contributors to pick up.

Member

I'll be opening this issue for other contributors to pick up.

SanchithHegde

added S-unactionable Status: There is not enough information to act on this problem S-unassigned Status: This issue has no one assigned to address it

and removed S-unactionable Status: There is not enough information to act on this problem

labels

Jul 4, 2023

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

Assignees

No one assigned

Labels
C-refactor Category: Refactor E-easy Effort: Should be easy to implement and would make a good first PR good first issue Good for newcomers help wanted Extra attention is needed P-low Priority: Low S-unassigned Status: This issue has no one assigned to address it
Projects

None yet

Milestone

No milestone

Development

No branches or pull requests

5 participants
and others

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK