nordicsemiconductor / asset-tracker-cloud-app-js Goto Github PK

See NordicSemiconductor/asset-tracker-cloud-docs#520

The current implementation has a number of problems

• first and foremost the overwhelming use of property drilling -> refactor to use custom hooks (see an example of this method use here)
• hardcoded UI design (this can be changed once hooks are used)
• uses outdated UI framework, #277
• has code for both supported cloud platforms, which is not relevant for users, and should be separated
• does only support one use case (pet tracker), but should in the future provide a UI / instructions for different nRF9160 application use cases
• very little to no tests, and it keeps breaking when libraries are merged that compile (TypeScript API did not change), but have bugs

Solves

• #446
• #298
• #277
• #82

New version:

• AWS: https://github.com/NordicSemiconductor/asset-tracker-cloud-app-aws-js

aws-amplify/amplify-ui#296 (comment)

to allow more convenient inspection of larger tracks.

https://getbootstrap.com/docs/5.0/migration/
reactstrap/reactstrap#1748
#428

See NordicSemiconductor/asset-tracker-cloud-docs@4d249c2

Currently the user has to reload the page.
Expiry of tokens should be handled more gracefully.

See NordicSemiconductor/asset-tracker-cloud-docs#521

Given a known history of max / min battery voltage for a device, calculate estimated time left until the battery runs out.

This shadow will crash the cat view:

{
  "reported": {
    "cfg": {
      "act": true,
      "gpst": 300,
      "actwt": 600,
      "mvres": 60,
      "mvt": 3600,
      "acct": 10
    },
    "gps": {
      "v": "$GPGGA,154821.59,6325.99925,N,01026.23650,E,1,04,2.06,80.51,M,0,,*13\r\n",
      "ts": 1623426501142
    },
    "bat": {
      "v": 5187,
      "ts": 1623426498299
    },
    "roam": {
      "v": {
        "rsrp": 49
      },
      "ts": 1623426498180
    }
  }
}

Related:

• https://devzone.nordicsemi.com/f/nordic-q-a/77063/cat-tracker-app-v9-0-13-whitescreen

AWS app has them.
Could be a UI or backend issue.

It should be possible for the user to select the time range of the charts and the map so they are comparable.
Right now every chart has it's own time range depending on the amount of messages, because most queries use LIMIT 100.

Add integration tests against a real AWS/Azure instance using Cypress or playwright to catch dependency incompatibilities that cause issues (like #297).

See NordicSemiconductor/asset-tracker-cloud-docs#568

See NordicSemiconductor/asset-tracker-cloud-docs#69 (comment)

Depends on NordicSemiconductor/asset-tracker-cloud-docs#175

See NordicSemiconductor/asset-tracker-cloud-docs#209

Add a new section to the device configuration with toggle buttons (like the Active/Passive one), for Firmware Modules that are Enabled by default, and if set to "Disabled", will add a string to the desired.nod array.

The two known modules for now are:

1. GPS (gnss)
2. Neighbor Cell Measurements (ncell)

If device reports "nod": ["gnss"] UI shows it as enabled.

Depends on bifravst/firmware#10

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Repository problems

These problems occurred while renovating this repository. View logs.

• WARN: Fallback to renovate.json file as a preset is deprecated, please use a default.json file instead.

Datasource	Name	Replacement PR?
npm	@babel/plugin-proposal-optional-chaining
npm	@types/react-leaflet
npm	@types/twemoji
npm	msal

Rate-Limited

These updates are currently rate-limited. Click on a checkbox below to force their creation now.

• chore(deps): update dependency @types/twemoji to v13.1.2
• chore(deps): update dependency eslint-plugin-import to v2.29.1
• chore(deps): update dependency husky to v9.0.11
• fix(deps): update dependency @amcharts/amcharts4 to v4.10.39
• fix(deps): update dependency @nordicsemiconductor/timestream-helpers to v6.0.2
• fix(deps): update dependency azure-iothub to v1.16.5
• fix(deps): update dependency fp-ts to v2.16.8
• fix(deps): update dependency io-ts to v2.2.21
• chore(deps): update dependency @commitlint/config-conventional to v18.6.3
• chore(deps): update dependency @types/lodash to v4.17.6
• chore(deps): update dependency @types/node to v20.14.10
• chore(deps): update dependency eslint-plugin-jsx-a11y to v6.9.0
• chore(deps): update dependency eslint-plugin-react to v7.34.3
• chore(deps): update react monorepo (@types/react, @types/react-dom, eslint-plugin-react-hooks)
• chore(deps): update dependency @commitlint/config-conventional to v19
• fix(deps): update dependency @microsoft/signalr to v8
• fix(deps): update dependency @nordicsemiconductor/rsrp-bar to v8
• fix(deps): update dependency date-fns to v3
• fix(deps): update dependency e118-iin-list to v4
• fix(deps): update dependency styled-components to v6
• fix(deps): update dependency use-debounce to v10
• fix(deps): update dependency uuid to v10 (uuid, @types/uuid)
• 🔐 Create all rate-limited PRs at once 🔐

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• fix(deps): update dependency @react-leaflet/core to v1.1.1
• chore(deps): update dependency @nordicsemiconductor/asset-tracker-cloud-code-style to v13
• chore(deps): update dependency react-scripts to v5
• fix(deps): update dependency @react-leaflet/core to v2
• fix(deps): update dependency intro.js to v7
• fix(deps): update dependency react-leaflet to v3.2.5 (react-leaflet, @types/react-leaflet)
• fix(deps): update dependency react-leaflet to v4
• fix(deps): update dependency react-router-dom to v6
• fix(deps): update dependency reactstrap to v9
• fix(deps): update react monorepo to v18 (major) (react, react-dom)
• Click on this checkbox to rebase all open PRs at once

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

See NordicSemiconductor/asset-tracker-cloud-docs#836

Right now the web application only listens to reported configuration changes, but does not listen to desired configuration changes.

See https://devzone.nordicsemi.com/f/nordic-q-a/74354/cat-tracker-in-multiple-tabs-or-windows

Right now the indication whether data is outdated is based on a fixed time. Instead, honour the configured active/passive timeouts.

Shadow state:

{
  "desired": {
    "cfg": {
      "gpst": 60,
      "mvres": 120,
      "mvt": 3600,
      "acct": 7.5,
      "actwt": 120
    }
  },
  "reported": {
    "bat": {
      "v": 5121,
      "ts": 1613651588962
    },
    "dev": {
      "v": {
        "band": 20,
        "nw": "LTE-M",
        "iccid": "89450421180216216095",
        "modV": "mfw_nrf9160_1.2.3",
        "brdV": "nrf9160dk_nrf9160",
        "appV": "0.0.0-development"
      },
      "ts": 1613650488083
    },
    "roam": {
      "v": {
        "rsrp": 66,
        "area": 2305,
        "mccmnc": 24202,
        "cell": 26902,
        "ip": "10.81.168.74"
      },
      "ts": 1613651588939
    },
    "cfg": {
      "act": true,
      "gpst": 60,
      "actwt": 120,
      "mvres": 120,
      "mvt": 3600,
      "acct": 7.5
    }
  }
}

In the web UI the location is located outside the coast of Africa. long: 0, lat: 0, alt: 0.

Replace jest with the Node.js test runner (and tsmatchers for assertions if needed).

v8.6.100 of the app does not work:

App tries to establish connect, but

fetch("https://assettrackerprodsignalr.service.signalr.net/client/negotiate?hub=deviceupdates&negotiateVersion=1", {
  "headers": {
    "accept": "*/*",
    "accept-language": "en-US,en;q=0.9,nb;q=0.8,de;q=0.7",
    "authorization": "Bearer eyJxxx0XA",
    "cache-control": "max-age=0",
    "content-type": "text/plain;charset=UTF-8",
    "sec-ch-ua": "\" Not A;Brand\";v=\"99\", \"Chromium\";v=\"90\", \"Google Chrome\";v=\"90\"",
    "sec-ch-ua-mobile": "?0",
    "sec-fetch-dest": "empty",
    "sec-fetch-mode": "cors",
    "sec-fetch-site": "cross-site",
    "x-requested-with": "XMLHttpRequest",
    "x-signalr-user-agent": "Microsoft SignalR/5.0 (5.0.6; Unknown OS; Browser; Unknown Runtime Version)"
  },
  "referrer": "https://assettrackerprodapp.z16.web.core.windows.net/",
  "referrerPolicy": "strict-origin-when-cross-origin",
  "body": "",
  "method": "POST",
  "mode": "cors"
});

returns

Connection count reaches limit.

The map already can become quite busy, and with the introduction of location data based on neighbor cell measurements, this will become worse.

Add a UI to configure which elements are visible.

One of the calls (probably cell geolocation) returns 404 but is parsed as JSON.

See NordicSemiconductor/asset-tracker-cloud-docs#590

Backend API: NordicSemiconductor/asset-tracker-cloud-azure-js#559

This has been implemented in the AWS flavour, hower not yet in the Azure flavour.

... to improve readability.

Add the changes from the AWS app to this app.