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

Retry logic for unblided tokens contribution #6940

Closed
NejcZdovc opened this issue Nov 15, 2019 · 1 comment · Fixed by brave/brave-core#4372
Closed

Retry logic for unblided tokens contribution #6940

NejcZdovc opened this issue Nov 15, 2019 · 1 comment · Fixed by brave/brave-core#4372
Assignees
Labels
feature/rewards priority/P2 A bad problem. We might uplift this to the next planned release. QA/No release-notes/exclude

Comments

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Nov 15, 2019

When we start contribution process we need to log all contribution data in the db. This should also include step and retry number. When we are going through publishers we need to log progress per publisher as well. If publisher submit fails we need to retry it.

@NejcZdovc NejcZdovc added feature/rewards priority/P2 A bad problem. We might uplift this to the next planned release. labels Nov 15, 2019
@NejcZdovc NejcZdovc self-assigned this Nov 15, 2019
@NejcZdovc
Copy link
Contributor Author

blocked on #6942

@NejcZdovc NejcZdovc removed the blocked label Nov 19, 2019
@NejcZdovc NejcZdovc removed their assignment Jan 8, 2020
@NejcZdovc NejcZdovc added this to the 1.3.x - Beta milestone Jan 14, 2020
@NejcZdovc NejcZdovc removed the blocked label Jan 14, 2020
@NejcZdovc NejcZdovc self-assigned this Jan 14, 2020
NejcZdovc added a commit to brave/brave-core that referenced this issue Jan 16, 2020
NejcZdovc added a commit to brave/brave-core that referenced this issue Jan 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/rewards priority/P2 A bad problem. We might uplift this to the next planned release. QA/No release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants