-
Notifications
You must be signed in to change notification settings - Fork 893
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
Adds redeemed_at to unblinded tokens, removes delete #5017
Conversation
a409e2b
to
d3043c6
Compare
@@ -86,77 +82,4 @@ TEST_F(UnblindedTest, NotEnoughFunds) { | |||
}); | |||
} | |||
|
|||
TEST_F(UnblindedTest, PromotionExpiredDeleteToken) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
deleted as we are not deleting tokens anymore
PR for updating DB schema and changelog https://github.com/brave/rewards/pull/6 |
@@ -34,6 +34,7 @@ index|sqlite_autoindex_sku_order_1|sku_order| | |||
index|sqlite_autoindex_sku_order_items_1|sku_order_items| | |||
index|sqlite_autoindex_sku_transaction_1|sku_transaction| | |||
index|unblinded_tokens_creds_id_index|unblinded_tokens|CREATE INDEX unblinded_tokens_creds_id_index ON unblinded_tokens (creds_id) | |||
index|unblinded_tokens_redeem_id_index|unblinded_tokens|CREATE INDEX unblinded_tokens_redeem_id_index ON unblinded_tokens (redeem_id) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we ever search on redeem_id
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we will in the future
@@ -728,7 +728,7 @@ class LedgerImpl : public ledger::Ledger { | |||
const std::string& order_id, | |||
ledger::GetSKUTransactionCallback callback); | |||
|
|||
virtual void GetUnblindedTokensByBatchTypes( | |||
virtual void GetSpendableUnblindedTokensByBatchTypes( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good name!
Resolves brave/brave-browser#8181
Submitter Checklist:
npm run lint
)git rebase master
(if needed).git rebase -i
to squash commits (if needed).Test Plan:
redeem*
fields are populated)Reviewer Checklist:
After-merge Checklist:
changes has landed on.