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

Adds Mobile Mapper Rewards HIP #590

Merged
merged 7 commits into from
Mar 22, 2023
Merged

Conversation

meowshka
Copy link
Contributor

No description provided.

@vincenzospaghetti
Copy link
Contributor

@meowshka this HIP is missing a few sections. Can you add:

Drawbacks

Why should we not do this?
What problems could occur if we do this?

Rationale and Alternatives

This is your chance to discuss your proposal in the context of the whole design space. This is probably the most important section!
Why is this design the best in the space of possible designs?
What other designs have been considered and what is the rationale for not choosing them?
What is the impact of not doing this?

Unresolved Questions

What parts of the design do you expect to resolve through the HIP process before this gets merged?
What parts of the design do you expect to resolve through the implementation of this feature?
What related issues do you consider out of scope for this HIP that could be addressed in the future independently of the solution that comes out of this HIP?
Are there dependencies, milestones, or dates that need to be met for this HIP to succeed?

Deployment Impact

Describe how this design will be deployed and any potential impact it may have on current users of this project.
How will current users be impacted?
How will existing documentation/knowledge base need to be supported? Any content to change at http://docs.helium.com/?
Is this backwards compatible? Can this HIP be undone?
If not, what is the procedure to migrate?

Success Metrics

What metrics can be used to measure the success of this design? Are any new ETL reports needed to measure the success?
What should we measure to prove a performance increase?
What should we measure to prove an improvement in stability?
What should we measure to prove a reduction in complexity?
What should we measure to prove an acceptance of this by its users?

@meowshka
Copy link
Contributor Author

@vincenzospaghetti done with latest commit c81e815

@vincenzospaghetti vincenzospaghetti merged commit 097fadc into main Mar 22, 2023
@hiptron hiptron deleted the meowshka/mappers_hip branch October 23, 2023 18:10
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.

2 participants