Scan, DB discovery library and hrs c sample. #514

eivindj-nordic · 2025-11-26T14:06:32Z

No description provided.

github-actions · 2025-11-26T14:08:05Z

You can find the documentation preview for this PR here.

peknis · 2025-12-01T06:44:44Z

doc/nrf-bm/libraries/bluetooth/ble_db_discovery.rst

+Bluetooth: Database Discovery library
+##############################


Suggested change

Bluetooth: Database Discovery library

##############################

Bluetooth: Database discovery library

#####################################

peknis · 2025-12-01T06:46:15Z

doc/nrf-bm/libraries/bluetooth/ble_db_discovery.rst

+   :local:
+   :depth: 2
+
+This module contains the APIs and types exposed by the Database Discovery module. These APIs and types can be used by the application to perform discovery of a service and its characteristics at the peer server. This module can also be used to discover the desired services in multiple remote devices.


Suggested change

This module contains the APIs and types exposed by the Database Discovery module. These APIs and types can be used by the application to perform discovery of a service and its characteristics at the peer server. This module can also be used to discover the desired services in multiple remote devices.

This module contains the APIs and types exposed by the Database discovery module.

These APIs and types can be used by the application to discover a service and its characteristics at the peer server.

This module can also be used to discover the desired services in multiple remote devices.

peknis · 2025-12-01T06:47:23Z

doc/nrf-bm/libraries/bluetooth/ble_db_discovery.rst

+Configuration
+*************
+
+The library is enabled and configured using the Kconfig system:


Suggested change

The library is enabled and configured using the Kconfig system:

To enable and configure the library use the following Kconfig options:

peknis · 2025-12-01T06:47:43Z

doc/nrf-bm/libraries/bluetooth/ble_db_discovery.rst

+==============
+
+The library is initialized by calling the :c:func:`ble_db_discovery_init` function.
+See the :c:struct:`ble_db_discovery_config` struct for configuration details.


Suggested change

See the :c:struct:`ble_db_discovery_config` struct for configuration details.

See the :c:struct:`ble_db_discovery_config` structure for configuration details.

peknis · 2025-12-01T06:49:31Z

doc/nrf-bm/libraries/bluetooth/ble_db_discovery.rst

+After initialization, use :c:func:`ble_db_discovery_service_register` to register a callback that triggers when a service is discovered with a specific UUID.
+To start discovering from a peer, you must use :c:func:`ble_db_discovery_start` with the connection handle of the peer device.
+For example, you can call :c:func:`ble_db_discovery_start` when the ble event :c:enum:`BLE_GAP_EVT_CONNECTED` is triggered and use the connection handle from the event as the second argument in :c:func:`ble_db_discovery_start`.


Suggested change

After initialization, use :c:func:`ble_db_discovery_service_register` to register a callback that triggers when a service is discovered with a specific UUID.

To start discovering from a peer, you must use :c:func:`ble_db_discovery_start` with the connection handle of the peer device.

For example, you can call :c:func:`ble_db_discovery_start` when the ble event :c:enum:`BLE_GAP_EVT_CONNECTED` is triggered and use the connection handle from the event as the second argument in :c:func:`ble_db_discovery_start`.

After initialization, use the :c:func:`ble_db_discovery_service_register` function to register a callback that triggers when a service is discovered with a specific UUID.

To start discovering from a peer, use the :c:func:`ble_db_discovery_start` function with the connection handle of the peer device.

For example, you can call the :c:func:`ble_db_discovery_start` function when the :c:enum:`BLE_GAP_EVT_CONNECTED` event is triggered and use the connection handle from the event as the second argument in the :c:func:`ble_db_discovery_start` function.

peknis · 2025-12-01T07:08:32Z

samples/bluetooth/ble_hrs_c/README.rst

+   If the device is not advertising, you might need to use the :guilabel:`Reset Board` option in |VSC|.
+#. :guilabel:`Connect` to your device.
+   The terminal output in |VSC| indicates ``Peer connected``.
+#. Observe that the services are shown in the connected device and that you can start receiving values for the Heart Rate and the Battery Service by clicking the Play button.


Suggested change

#. Observe that the services are shown in the connected device and that you can start receiving values for the Heart Rate and the Battery Service by clicking the Play button.

#. Observe that the services are shown in the connected device and you can start receiving values for the Heart Rate and the Battery Service when you click the :guilabel:`Play` button.

peknis · 2025-12-01T07:08:59Z

samples/bluetooth/ble_hrs_c/README.rst

+#. In nRF Connect for Desktop, scan for advertising devices.
+   Your device should be advertising as ``nRF_BM_HRS``.
+   If the device is not advertising, you might need to use the :guilabel:`Reset Board` option in |VSC|.
+#. :guilabel:`Connect` to your device.


Suggested change

#. :guilabel:`Connect` to your device.

#. Click :guilabel:`Connect` to connect your device.

peknis · 2025-12-01T07:09:38Z

samples/bluetooth/ble_hrs_c/README.rst

+#. :guilabel:`Connect` to your device.
+   The terminal output in |VSC| indicates ``Peer connected``.
+#. Observe that the services are shown in the connected device and that you can start receiving values for the Heart Rate and the Battery Service by clicking the Play button.
+   Heart Rate notifications are received every second, and Battery Level notifications are received every two seconds.


Suggested change

Heart Rate notifications are received every second, and Battery Level notifications are received every two seconds.

Heart Rate notifications are received every second, and Battery Level notifications every two seconds.

peknis · 2025-12-01T07:10:41Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+Otherwise, the :c:macro:`NRF_BLE_SCAN_EVT_NOT_FOUND` event is generated.
+
+You can enable filters by calling the :c:func:`ble_scan_filters_enable` function after initialization.
+You can activate filters for one filter type, or for a combination of several filter types.


Suggested change

You can activate filters for one filter type, or for a combination of several filter types.

You can activate filters for one filter type, or for a combination of several filter types.

Example code:

peknis · 2025-12-01T07:11:35Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+=======================
+
+The advanced initialization provides a larger configuration set for the library.
+It is required when using scan filters or the whitelist.


Suggested change

It is required when using scan filters or the whitelist.

It is required when using scan filters or the whitelist.

Example code:

nordicjm · 2025-12-01T08:12:23Z

doc/nrf-bm/doxygen/nrf-bm.doxyfile


 PREDEFINED             = __DOXYGEN__ \
                         CONFIG_BLE_QWR_MAX_ATTR=1 \
+                         CONFIG_BLE_SCAN_FILTER=1 \


why? We don't do this in zephyr and we don't do this in NCS, so why is this required in BM?

It is required until we have #401 I think. After that doc generation will be the same as in NCS.

nordicjm · 2025-12-01T08:13:16Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+* :kconfig:option:`CONFIG_BLE_SCAN_INTERVAL` - Determines the scan interval in units of 0.625 ms.
+* :kconfig:option:`CONFIG_BLE_SCAN_DURATION` - Duration of a scanning session in units of 10 ms, if set to 0, the scanning continues until it is explicitly disabled.
+* :kconfig:option:`CONFIG_BLE_SCAN_WINDOW` - Determines the scanning window in units of 0.625 ms.
+* :kconfig:option:`CONFIG_BLE_SCAN_SLAVE_LATENCY` - Determines the slave latency in counts of connection events.


peripheral latency, as per BT SIG, update Kconfigs and names

@hermabe The SoftDevice uses slave latency, I assume that is for legacy reasons (avoid breaking API)?

Yes, the SoftDevice still refers to pre-5.3 naming in many places. We decided early on to not rename everything to not break the API.

You can call the Kconfig whatever you want. I am more concerned with why these are kconfigs at all, shouldnt the connection parameters be runtime configurable?

These were part of the sdk_config.h in nRF5. The ble_gap_scan_params_t provided are still configurable through ble_scan_params_set().

nordicjm · 2025-12-01T08:13:33Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+
+.. note::
+
+   When using the whitelist, filters are inactive.


as other PR, allow list

lib/bluetooth/ble_scan/Kconfig

nordicjm

"HRS_C" Kconfig name is awful and non descriptive, use full word "HRS_CENTRAL"

nordicjm · 2025-12-01T09:32:57Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+                   .filter_policy = BLE_GAP_SCAN_FP_WHITELIST,
+                   .timeout = SCAN_DURATION_WHITELIST,


these need updated

Top one is from SoftDevice, cannot be changed.

nordicjm · 2025-12-01T09:33:08Z

lib/bluetooth/ble_scan/Kconfig

+	  Determines the scanning window in units of 0.625 milliseconds.
+
+config BLE_SCAN_PERIPHERAL_LATENCY
+	int "Scan slave latency"


needs update

nordicjm · 2025-12-01T09:34:21Z

lib/bluetooth/ble_db_discovery/Kconfig

+	range 1 6
+	default 6
+	help
+		The maximum number of database characteristics.


wrong indent for all these

nordicjm · 2025-12-01T09:34:55Z

lib/bluetooth/ble_db_discovery/Kconfig

+	depends on BLE_GATT_QUEUE
+	select EXPERIMENTAL
+	help
+	  database discovery.


if this is going to be the help text, might as well remove it. Either that or expand it here properly, with capital first letter

nordicjm · 2025-12-01T09:35:04Z

lib/bluetooth/ble_db_discovery/Kconfig

+	help
+		The total number of services that can be discovered by this module.
+
+config SRV_DISC_START_HANDLE


missing prefix

nordicjm · 2025-12-01T09:35:19Z

tests/lib/bluetooth/ble_db_discovery/CMakeLists.txt

+        SVCALL_AS_NORMAL_FUNCTION=1
+        NRF54L15_XXAA)
+
+target_compile_definitions( app PRIVATE


looks like the old way?

Yes, updated locally.

nordicjm · 2025-12-01T09:36:20Z

samples/bluetooth/ble_hrs_c/Kconfig

+
+menu "BLE HRS Central sample"
+
+config APP_USE_TARGET_PERIPH_NAME


*PERIPHERAL, these Kconfig names are not long, they can use full words

nordicjm · 2025-12-01T09:36:33Z

samples/bluetooth/ble_hrs_c/Kconfig

+endif # APP_USE_TARGET_PERIPH_ADDR
+
+module=APP_BLE_HRS_C_SAMPLE
+module-dep=LOG


Suggested change

module-dep=LOG

nordicjm · 2025-12-01T09:36:41Z

samples/bluetooth/ble_hrs_c/Kconfig

+module-str=BLE Heart Rate Central Service Sample
+source "$(ZEPHYR_BASE)/subsys/logging/Kconfig.template.log_config"
+
+endmenu # "BLE HRS sample"


nordicjm · 2025-12-01T09:36:48Z

samples/bluetooth/ble_hrs_c/Kconfig

+# SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
+#
+
+menu "BLE HRS Central sample"


nordicjm · 2025-12-01T10:35:01Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+                   .active = 0x01,
+                   .interval = NRF_BLE_SCAN_INTERVAL,
+                   .window = NRF_BLE_SCAN_WINDOW,
+                   .filter_policy = BLE_GAP_SCAN_FP_WHITELIST,


BLE_GAP_SCAN_FP_WHITELIST is still not updated, this name is not acceptable

It cannot be updated here. This is part of the SoftDevice deliverables.

sdk-nrf-bm/components/softdevice/s145/s145_API/include/ble_gap.h

Line 454 in f0a003f

#define BLE_GAP_SCAN_FP_WHITELIST 0x01 /**< Accept advertising packets from devices in the whitelist except directed

See #514 (comment)

peknis · 2025-12-01T10:47:42Z

doc/nrf-bm/libraries/bluetooth/ble_db_discovery.rst

+Configuration
+*************
+
+To enable and configure the library use the following Kconfig options:


Suggested change

To enable and configure the library use the following Kconfig options:

To enable and configure the library, use the following Kconfig options:

Sorry, my bad forgetting the comma.

peknis · 2025-12-01T10:49:22Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+
+The allow list (formerly known as whitelist) stores information about all the device connections and bonding.
+If you enable the allow list, the application receives advertising packets only from the devices that are on the allow list.
+An advertising package from a allow listed device generates an :c:macro:`BLE_SCAN_EVT_ALLOW_LIST_ADV_REPORT` event.


Suggested change

An advertising package from a allow listed device generates an :c:macro:`BLE_SCAN_EVT_ALLOW_LIST_ADV_REPORT` event.

An advertising package from an allow-listed device generates an :c:macro:`BLE_SCAN_EVT_ALLOW_LIST_ADV_REPORT` event.

peknis · 2025-12-01T10:50:56Z

doc/nrf-bm/libraries/bluetooth/ble_scan.rst

+* Multifilter - At least one filter from each filter type you set must be matched to generate an event.
+   For UUID filters, all specified UUIDs must match in this mode.
+   To enabled multifilter, set the :c:macro:`match_all` argument to true when calling the :c:func:`ble_scan_filters_enable` function.


Suggested change

* Multifilter - At least one filter from each filter type you set must be matched to generate an event.

For UUID filters, all specified UUIDs must match in this mode.

To enabled multifilter, set the :c:macro:`match_all` argument to true when calling the :c:func:`ble_scan_filters_enable` function.

* Multifilter - At least one filter from each filter type you set must be matched to generate an event.

For UUID filters, all specified UUIDs must match in this mode.

To enabled multifilter, set the :c:macro:`match_all` argument to true when calling the :c:func:`ble_scan_filters_enable` function.

peknis · 2025-12-01T10:52:40Z

include/bm/bluetooth/services/ble_hrs_c.h

+uint32_t ble_hrs_c_init(struct ble_hrs_c *ble_hrs_c, struct ble_hrs_c_config *ble_hrs_c_init);
+
+/**
+ * @brief Handle BLE events from the SoftDevice.


Suggested change

* @brief Handle BLE events from the SoftDevice.

* @brief Handle Bluetooth LE events from the SoftDevice.

peknis · 2025-12-01T10:53:12Z

include/bm/bluetooth/services/ble_hrs_c.h

+ * @details This function handles the BLE events received from the SoftDevice. If a BLE
+ *          event is relevant to the Heart Rate Client module, the function uses the


Suggested change

* @details This function handles the BLE events received from the SoftDevice. If a BLE

* event is relevant to the Heart Rate Client module, the function uses the

* @details This function handles the Bluetooth LE events received from the SoftDevice. If an

* event is relevant to the Heart Rate Client module, the function uses the

peknis · 2025-12-01T10:53:28Z

include/bm/bluetooth/services/ble_hrs_c.h

+ * event's data to update interval variables and, if necessary, send events to the
+ * application.
+ *
+ * @param[in] ble_evt Pointer to the BLE event.


Suggested change

* @param[in] ble_evt Pointer to the BLE event.

* @param[in] ble_evt Pointer to the Bluetooth LE event.

peknis · 2025-12-01T10:54:22Z

lib/bluetooth/ble_scan/Kconfig

+#
+
+menuconfig BLE_SCAN
+	bool "BLE Scan [EXPERIMENTAL]"


Suggested change

bool "BLE Scan [EXPERIMENTAL]"

bool "Bluetooth LE Scan [EXPERIMENTAL]"

peknis · 2025-12-01T10:56:03Z

subsys/bluetooth/services/ble_hrs_central/Kconfig

+#
+
+menuconfig BLE_HRS_CENTRAL
+	bool "BLE Heart rate service central [EXPERIMENTAL]"


Suggested change

bool "BLE Heart rate service central [EXPERIMENTAL]"

bool "Bluetooth LE Heart rate service central [EXPERIMENTAL]"

carlescufi · 2025-12-01T13:26:57Z

"HRS_C" Kconfig name is awful and non descriptive, use full word "HRS_CENTRAL"

Agreed, and I think we should use hrs_central_* everywhere IMHO.

eivindj-nordic · 2025-12-01T13:39:13Z

"HRS_C" Kconfig name is awful and non descriptive, use full word "HRS_CENTRAL"

Agreed, and I think we should use hrs_central_* everywhere IMHO.

Ok, I'll make the changes.

nordicjm · 2025-12-01T13:58:46Z

include/bm/bluetooth/ble_db_discovery.h

+ *                          has been reached.
+ */
+uint32_t ble_db_discovery_service_register(struct ble_db_discovery *db_discovery,
+				       const ble_uuid_t *const uuid);


nit. alignment is off

nordicjm · 2025-12-01T13:59:13Z

include/bm/bluetooth/services/ble_hrs_central.h

@@ -0,0 +1,242 @@
+/*


commit title is wrong

Updated locally.

nordicjm · 2025-12-01T13:59:56Z

include/bm/bluetooth/services/ble_hrs_central.h

+/**
+ * @brief Heart Rate Measurement received from the peer.
+ */
+struct ble_hrm {


does this conflict if you include the peripheral and central files in a .c file?

This is only defined in ble_hrs_central.h.
ble_hrs.h defines heart rate service (hrs), not heart rate measurement (hrm).

nordicjm · 2025-12-01T14:00:22Z

samples/bluetooth/ble_hrs_central/CMakeLists.txt

+cmake_minimum_required(VERSION 3.20.0)
+
+find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
+project(ble_hrs)


Suggested change

project(ble_hrs)

project(ble_hrs_central)

Add scan library. Co-authored-by: Sondre Pettersen <[email protected]> Co-authored-by: Asil Zogby <[email protected]> Signed-off-by: Eivind Jølsgard <[email protected]>

Add db_discovery library. Co-authored-by: Sondre Pettersen <[email protected]> Co-authored-by: Asil Zogby <[email protected]> Signed-off-by: Eivind Jølsgard <[email protected]>

Add BLE heart rate service central and sample. Co-authored-by: Asil Zogby <[email protected]> Signed-off-by: Sondre Pettersen <[email protected]> Signed-off-by: Eivind Jølsgard <[email protected]>

github-actions bot added the doc-required PR must not be merged without tech writer approval. label Nov 26, 2025

eivindj-nordic added this to the v1.0.0 milestone Nov 27, 2025

eivindj-nordic force-pushed the hrs_central branch from 27651ce to ccc67f8 Compare November 27, 2025 15:52

anhmolt force-pushed the hrs_central branch 2 times, most recently from f1b6f15 to 345c14e Compare November 28, 2025 11:11

eivindj-nordic force-pushed the hrs_central branch 4 times, most recently from def7c1f to ff5b64e Compare November 28, 2025 13:38

eivindj-nordic marked this pull request as ready for review November 28, 2025 13:58

eivindj-nordic requested review from a team as code owners November 28, 2025 13:58

peknis requested changes Dec 1, 2025

View reviewed changes

nordicjm requested changes Dec 1, 2025

View reviewed changes

eivindj-nordic mentioned this pull request Dec 1, 2025

Add scan library #446

Closed

eivindj-nordic changed the title ~~DB discovery library and hrs c sample.~~ Scan, DB discovery library and hrs c sample. Dec 1, 2025

eivindj-nordic force-pushed the hrs_central branch 2 times, most recently from 1427e26 to 28eb23a Compare December 1, 2025 09:30

nordicjm requested changes Dec 1, 2025

View reviewed changes

eivindj-nordic force-pushed the hrs_central branch from 28eb23a to 5c0c3ca Compare December 1, 2025 10:00

nordicjm reviewed Dec 1, 2025

View reviewed changes

peknis requested changes Dec 1, 2025

View reviewed changes

eivindj-nordic force-pushed the hrs_central branch 2 times, most recently from c153d9c to 9acf427 Compare December 1, 2025 12:11

eivindj-nordic requested review from nordicjm and peknis December 1, 2025 12:16

peknis approved these changes Dec 1, 2025

View reviewed changes

eivindj-nordic force-pushed the hrs_central branch 2 times, most recently from 5e6dae8 to fd67120 Compare December 1, 2025 13:57

nordicjm reviewed Dec 1, 2025

View reviewed changes

eivindj-nordic and others added 2 commits December 1, 2025 15:06

lib: scan: add scan library

4e14e88

Add scan library. Co-authored-by: Sondre Pettersen <[email protected]> Co-authored-by: Asil Zogby <[email protected]> Signed-off-by: Eivind Jølsgard <[email protected]>

lib: db_discovery: add database discovery library

7ae9f0b

Add db_discovery library. Co-authored-by: Sondre Pettersen <[email protected]> Co-authored-by: Asil Zogby <[email protected]> Signed-off-by: Eivind Jølsgard <[email protected]>

eivindj-nordic force-pushed the hrs_central branch 2 times, most recently from bfc6b15 to 909a78f Compare December 1, 2025 14:08

bluetooth: services: ble_hrs_central: add heart rate service central

8fa7345

Add BLE heart rate service central and sample. Co-authored-by: Asil Zogby <[email protected]> Signed-off-by: Sondre Pettersen <[email protected]> Signed-off-by: Eivind Jølsgard <[email protected]>

eivindj-nordic force-pushed the hrs_central branch from 909a78f to 8fa7345 Compare December 1, 2025 14:09

nordicjm approved these changes Dec 1, 2025

View reviewed changes

anhmolt approved these changes Dec 1, 2025

View reviewed changes

eivindj-nordic merged commit b251cd5 into nrfconnect:main Dec 1, 2025
18 checks passed

		Bluetooth: Database Discovery library
		##############################

	The library is enabled and configured using the Kconfig system:
	To enable and configure the library use the following Kconfig options:

	See the :c:struct:`ble_db_discovery_config` struct for configuration details.
	See the :c:struct:`ble_db_discovery_config` structure for configuration details.

	#. Observe that the services are shown in the connected device and that you can start receiving values for the Heart Rate and the Battery Service by clicking the Play button.
	#. Observe that the services are shown in the connected device and you can start receiving values for the Heart Rate and the Battery Service when you click the :guilabel:`Play` button.

	#. :guilabel:`Connect` to your device.
	#. Click :guilabel:`Connect` to connect your device.

	Heart Rate notifications are received every second, and Battery Level notifications are received every two seconds.
	Heart Rate notifications are received every second, and Battery Level notifications every two seconds.

	You can activate filters for one filter type, or for a combination of several filter types.
	You can activate filters for one filter type, or for a combination of several filter types.
	Example code:

	It is required when using scan filters or the whitelist.
	It is required when using scan filters or the whitelist.
	Example code:

		.filter_policy = BLE_GAP_SCAN_FP_WHITELIST,
		.timeout = SCAN_DURATION_WHITELIST,


		menu "BLE HRS Central sample"

		config APP_USE_TARGET_PERIPH_NAME

	To enable and configure the library use the following Kconfig options:
	To enable and configure the library, use the following Kconfig options:

	An advertising package from a allow listed device generates an :c:macro:`BLE_SCAN_EVT_ALLOW_LIST_ADV_REPORT` event.
	An advertising package from an allow-listed device generates an :c:macro:`BLE_SCAN_EVT_ALLOW_LIST_ADV_REPORT` event.

	* @brief Handle BLE events from the SoftDevice.
	* @brief Handle Bluetooth LE events from the SoftDevice.

		* @details This function handles the BLE events received from the SoftDevice. If a BLE
		* event is relevant to the Heart Rate Client module, the function uses the

	* @param[in] ble_evt Pointer to the BLE event.
	* @param[in] ble_evt Pointer to the Bluetooth LE event.

	bool "BLE Scan [EXPERIMENTAL]"
	bool "Bluetooth LE Scan [EXPERIMENTAL]"

	bool "BLE Heart rate service central [EXPERIMENTAL]"
	bool "Bluetooth LE Heart rate service central [EXPERIMENTAL]"

Scan, DB discovery library and hrs c sample. #514

Scan, DB discovery library and hrs c sample. #514

Uh oh!

Conversation

eivindj-nordic commented Nov 26, 2025

Uh oh!

github-actions bot commented Nov 26, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nordicjm left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!