Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Current Timestamp as the transaction number instead of increasing by 1 from the last transaction number #2137

Open
velmuruganc opened this issue Jan 25, 2025 · 5 comments
Labels

Comments

@velmuruganc
Copy link
Contributor

We get different transaction numbers from different sources. I mean different number series for each bank. The current behavior is taking the largest number from the db and adding 1.

For Example, I received two transaction SMS from bank and one ref no is "23412890" and another is "128976872". When try to and another txn and click +1 btn, I will get txn no. as "128976873". In place of this, I would prefer to have timestamp as txn number for all manual txns

Image
@velmuruganc
Copy link
Contributor Author

velmuruganc commented Jan 25, 2025

@wolfsolver , I hope you understand the requirements. I will check-in my changes once you merge my 1st PR.

@wolfsolver
Copy link
Member

Your PR was merged this morning.
Feel free to go ahead.

@velmuruganc
Copy link
Contributor Author

@wolfsolver, why i am not able to create new branch from master. I don't see an option. Do i need to get different access? If so please provide.

@wolfsolver
Copy link
Member

You can create a branch from your repository...
Not from money manager repo, because you are not in the team

@guanlisheng
Copy link
Contributor

hi @velmuruganc

let's chat more about this behavior change. currently, the transaction number is optional per account and starts from 1. It is much different from a timestamp. we have the transaction date to store the timestamp information

in the meanwhile, i will invite you to join the team, then you can work on the feature branch in the main repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants