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

subscriber location rewards #530

Merged
merged 1 commit into from
Jun 27, 2023
Merged

Conversation

andymck
Copy link
Contributor

@andymck andymck commented Jun 6, 2023

Linked proto PR:

helium/proto#345

Adds support for subscriber location rewards

Linked HIP: https://github.com/helium/HIP/blob/main/0079-mobile-poc-mappers-rewards.md
Linked issue: #422

TODO:

  • plug in carrier key API usage, when API is available
  • remove local debug TODOs
  • merge proto 345 and update deps
  • squash commits

Tested locally end to end, example of subscriber reward outputted :

{
"start_period": 1686052372,
"end_period": 1686138856,
"reward": {
"SubscriberReward": {
"subscriber_id": [
115,
117,
98,
115,
99,
114,
105,
98,
101,
114,
95,
120
],
"discovery_location_amount": 30000000
}
}
}

@andymck andymck mentioned this pull request Jun 6, 2023
2 tasks
@andymck andymck force-pushed the andymck/location-sharing-rewards branch from 0bf3ccb to f1b17b2 Compare June 6, 2023 15:17
mobile_verifier/migrations/13_data_session.sql Outdated Show resolved Hide resolved
mobile_verifier/migrations/14_subscriber_location.sql Outdated Show resolved Hide resolved
mobile_verifier/src/cli/server.rs Show resolved Hide resolved
reward_index/migrations/6_add_subscriber_reward_type.sql Outdated Show resolved Hide resolved
reward_index/src/indexer.rs Outdated Show resolved Hide resolved
mobile_verifier/src/rewarder.rs Outdated Show resolved Hide resolved
mobile_verifier/src/subsciber_location.rs Outdated Show resolved Hide resolved
mobile_verifier/src/subsciber_location.rs Outdated Show resolved Hide resolved
mobile_verifier/src/subsciber_location.rs Outdated Show resolved Hide resolved
mobile_verifier/src/subsciber_location.rs Outdated Show resolved Hide resolved
@andymck andymck force-pushed the andymck/location-sharing-rewards branch 2 times, most recently from feb806f to 2614848 Compare June 13, 2023 10:27
@andymck andymck force-pushed the andymck/location-sharing-rewards branch 2 times, most recently from 0043ed0 to 61561b7 Compare June 17, 2023 11:37
@andymck andymck marked this pull request as ready for review June 20, 2023 09:29
file_store/src/cli/dump.rs Outdated Show resolved Hide resolved
file_store/src/mobile_subscriber.rs Show resolved Hide resolved
file_store/src/mobile_subscriber.rs Show resolved Hide resolved
mobile_verifier/migrations/14_subscriber_location.sql Outdated Show resolved Hide resolved
mobile_verifier/src/settings.rs Outdated Show resolved Hide resolved
mobile_verifier/src/data_session.rs Outdated Show resolved Hide resolved
mobile_verifier/src/data_session.rs Outdated Show resolved Hide resolved
mobile_verifier/src/reward_shares.rs Outdated Show resolved Hide resolved
mobile_verifier/src/rewarder.rs Show resolved Hide resolved
@andymck andymck force-pushed the andymck/location-sharing-rewards branch from c12472c to 8be98a6 Compare June 27, 2023 15:11
@andymck andymck merged commit 91ae5a5 into main Jun 27, 2023
@andymck andymck deleted the andymck/location-sharing-rewards branch June 27, 2023 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants