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

[Feature] Scan to assert antenna gain and elevation #976

Open
rawrmaan opened this issue Oct 17, 2021 · 0 comments
Open

[Feature] Scan to assert antenna gain and elevation #976

rawrmaan opened this issue Oct 17, 2021 · 0 comments
Milestone

Comments

@rawrmaan
Copy link
Contributor

With the upcoming PoCv11 rollout, it's advised that all hotspot owners assert their antenna elevation and gain on-chain. This applies to fleet owners as well. Currently, the best-case (most efficient) way to do this for fleet owners is as follows:

  1. Display a QR code with the string helium://hotspot/[address] to your deployers
  2. Have them scan the QR code to go to the hotspot's page in the app
  3. They press the Settings button, then Update Hotspot
  4. They choose Custom Antenna from the list and manually enter the hotspot's gain and elevation
  5. They press Submit

Proposal

Add the ability to scan elevation and gain and tap "I Confirm", completing the entire process in two steps. This could be accomplished in either of the following ways:

  • Amend the existing scan-to-assert view and hotspot_location scan type to accept elevation and gain parameters in the QR code, keeping in mind that all 3 of the following scenarios will now be possible and should be accounted for:
    -- Scan hotspotAddress, latitude, longitude, elevation and gain all at once
    -- Scan hotspotAddress, latitude and longitude only (this is what's current supported)
    -- Scan hotspotAddress, elevation and gain only
  • Add a new scan type hotspot_antenna_info which accepts hotspotAddress, elevation and gain only and create a new UI to go along with it. Downside of this is that it is technically possible to assert location and antenna info all in one assert_location_v2 transaction to save time and fees, and this would not support that.

cc @MichaelBrew

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

No branches or pull requests

2 participants