Add option to navigate to new trace from resultscreen #597

puneet-pdx · 2024-09-30T20:54:54Z

Related to issue: # https://devtopia.esri.com/runtime/kotlin/issues/4492

Description:

adds functionality to go from ResultScreen to newTrace and from newTrace back to ResultScreen if results are available

Summary of changes:

Add logic to TraceState to clear trace parameters once a successful trace is done
Add TabRow to go between ResultScreen and TraceOptionScreen

Pre-merge Checklist

a vTest Job for this PR has been run
- link: ~~https://runtime-kotlin.esri.com/view/all/job/vtest/job/toolkit/120/~~
- https://runtime-kotlin.esri.com/view/all/job/vtest/job/toolkit/128/
Unit and/or integration tests have been added to exercise this PR's logic, and the tests are passing:
- Yes
- No

…NavigateToNewTrace # Conflicts: # toolkit/utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceNavigation.kt # toolkit/utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceOptionsScreen.kt

sorenoid

Looks good, I have some suggestions.

sorenoid · 2024-10-02T20:21:18Z

toolkit/utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/TraceState.kt

        return true
    }

+    private fun resetCurrentTrace() {
+        _selectedTraceConfiguration.value = null
+        _currentTraceStartingPoints.clear()


the design spec has a user story that says As a dispatcher comparing incident reports, I want to be able to perform analysis on a new trace type without losing the specified starting points.

I'd interpret this as meaning the starting points should remain between traces.

sorenoid · 2024-10-02T20:22:10Z

toolkit/utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/TraceState.kt

+        _currentTraceStartingPoints.clear()
+        _currentTraceName.value = ""
+        currentTraceGraphicsColor = Color.green
+        _currentTraceZoomToResults.value = false


the design spec also says By default, the control will zoom to the trace result area. This feature can be disabled by setting the AutoZoomToTraceResults property of the control to false. which I interpret as meaning this value should be true by default.

sorenoid · 2024-10-02T20:25:07Z

...tilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceOptionsScreen.kt

+@Composable
+private fun TabRow(onBackToResults: () -> Unit) {
+    val tabItems = listOf("New Trace", "Results")
+    var selectedTabIndex by remember { mutableIntStateOf(0) }


Suggested change

var selectedTabIndex by remember { mutableIntStateOf(0) }

var selectedTabIndex by rememberSaveable { mutableIntStateOf(0) }

sorenoid · 2024-10-02T20:31:45Z

...tilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceOptionsScreen.kt

@@ -161,6 +176,29 @@ internal fun TraceOptionsScreen(
    }
 }

+@Composable
+private fun TabRow(onBackToResults: () -> Unit) {
+    val tabItems = listOf("New Trace", "Results")


these should be localized

sorenoid · 2024-10-02T20:49:20Z

toolkit/utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/TraceState.kt

@@ -204,7 +206,7 @@ public class TraceState(

    internal fun setSelectedTraceConfiguration(config: UtilityNamedTraceConfiguration) {
        _selectedTraceConfiguration.value = config
-        _currentTraceName.value = "${config.name} ${(completedTraces.count { it.configuration.name == config.name } + 1)}"
+        _currentTraceName.value = "${config.name} ${(_completedTraces.count { it.configuration.name == config.name } + 1)}"


If I set the name in advanced options before running the trace, this is overwriting it.

I would set the currentTraceName to the default value at initialization and in resetCurrentTrace, then just not do this here at all.

as per our discussion this is working as expected.

sorenoid · 2024-10-02T21:11:12Z

...tilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceOptionsScreen.kt

+            var selected by remember { mutableStateOf(index == 0) }
+            Tab(
+                selected = selected,
+                onClick = {
+                    selectedTabIndex = index
+                    selected = true
+                    if (index == 1) {
+                        onBackToResults()
+                    }
+                },


Suggested change

var selected by remember { mutableStateOf(index == 0) }

Tab(

selected = selected,

onClick = {

selectedTabIndex = index

selected = true

if (index == 1) {

onBackToResults()

}

},

Tab(

selected = index == selectedTabIndex,

onClick = {

selectedTabIndex = index

if (index == 1) {

onBackToResults()

}

},

sorenoid · 2024-10-02T21:13:17Z

...utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceResultScreen.kt

            }
        }
    }
 }

+@Composable


same advice as above. Also, it seems like the internalTabRow could be reused in both results and trace options by passing in the lambda, tabItems, and selected index.

sorenoid

Looking good, I have a couple more questions/suggestions.

sorenoid · 2024-10-04T17:30:52Z

...utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/internal/util/TabRow.kt

+) {
+    var selectedTabIndex by rememberSaveable { mutableIntStateOf(selectedIndex) }
+
+    androidx.compose.material3.TabRow(selectedTabIndex = selectedTabIndex) {


why is this fully qualified? It is because this function has the same name? If that is the case I would change the name of this function.

Not needed. IDE brought it in on copy paste. The import works

sorenoid · 2024-10-04T17:32:16Z

...utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/internal/util/TabRow.kt

+@Composable
+internal fun TabRow(
+    onTabSelected: () -> Unit,
+    tabItems: List<String> = listOf(stringResource(R.string.new_trace), stringResource(R.string.results)),


this doesn't seem to need to be a parameter. If you want to keep it a parameter, I suggest moving it to the end of the parameter list so that selectedIndex doesn't have to be explicitly named when calling this function.

sorenoid

LGTM, two more minor suggestions.

sorenoid · 2024-10-04T18:16:56Z

...tilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceOptionsScreen.kt

@@ -118,7 +118,7 @@ internal fun TraceOptionsScreen(
            horizontalAlignment = Alignment.CenterHorizontally) {

            if (showResultsTab) {
-                TabRow(onBackToResults)
+                TabRow(onBackToResults, selectedIndex = 0)


Suggested change

TabRow(onBackToResults, selectedIndex = 0)

TabRow(onBackToResults, 0)

sorenoid · 2024-10-04T18:17:34Z

...utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceResultScreen.kt

@@ -79,7 +79,7 @@ internal fun TraceResultScreen(
            .fillMaxSize()
            .padding(horizontal = 10.dp)) {

-            TabRow(onBackToNewTrace)
+            TabRow(onBackToNewTrace, selectedIndex = 1)


Suggested change

TabRow(onBackToNewTrace, selectedIndex = 1)

TabRow(onBackToNewTrace, 1)

eri9000

LGTM
A few suggestions but otherwise 👍🏼

eri9000 · 2024-10-07T20:45:20Z

toolkit/utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/TraceState.kt

@@ -22,6 +22,7 @@ import androidx.compose.runtime.Immutable
 import androidx.compose.runtime.MutableState
 import androidx.compose.runtime.Stable
 import androidx.compose.runtime.State
+import androidx.compose.runtime.mutableIntStateOf


unused import

eri9000 · 2024-10-07T21:14:44Z

...utilitynetworks/src/main/java/com/arcgismaps/toolkit/utilitynetworks/ui/TraceResultScreen.kt

+            Spacer(
+                modifier = Modifier
+                    .fillMaxWidth()
+                    .height(15.dp)
+            )


Would suggest moving this to TabRow as you do the same in both Result screen and Options screen

Reading this in the code here makes more sense, as we are adding the tabrow and the tabrow should not be adding additional space after it implicitly.

* WIP * reset current trace * WIP * update padding values * remove newline * address code review comments * refactor tabrow * add newline * address code review comments * remove explicit name * remove unused import

@mit10976

* Sor10874/un trace tool (#540) * add UtilityNetwork components and Trace microapp modules * Upgrades to Kotlin 2.0 and K2 compiler and latest API build (#538) * barebones * whitespace * secrets defaults * add README and default Modifier parameter. update API file. * feedback on naming. * update package name * use compose compiler plugin --------- Co-authored-by: Soren Roth <sor10874@esri.com> Co-authored-by: Erick Lopez Solis <erick_solis@esri.com> * Barebones implementation (#543) * Changes to perform Trace with hardcoded inputs * refactor * optimize imports * Initial Implementation of UN Trace tool UI(#544) * add theming to ExpandableCard (#548) * add theming to ExpandableCard * add use of composition local shapes to ExpandableCard header * feedback * remove unused typography style. feedback. * add new configurable styles. make theme internal. other feedback --------- Co-authored-by: Soren Roth <sor10874@esri.com> * Utility network trace design state obj (#554) * Changes to add Trace state * Strip down implementation * UN trace tool config UI (#552) * add theming to ExpandableCard * wip * add use of composition local shapes to ExpandableCard header * wip * UI standardized and functionally complete. * bring experimental compose foundation UI into toolkit. Out of experimental in v1.6.6 (compose BOM 2024.08.00), can undo when we rev. * feedback * add keys to remember. fix concurrent modification of list. * remove print statement * remove print statement * remove unused typography style. feedback. * add new configurable styles. make theme internal. other feedback * comments * add comments to InternalMutatorMutex * PR feedback * whitespace * fix internal ExpandableCardHeader padding by adding it to the customizable theme. * center the add new points button * StartingPointRow -> StartingPoint * whitespace --------- Co-authored-by: Soren Roth <sor10874@esri.com> * update api file (#558) Co-authored-by: Soren Roth <sor10874@esri.com> * advanced options ui (#557) * add theming to ExpandableCard * wip * add use of composition local shapes to ExpandableCard header * wip * UI standardized and functionally complete. * bring experimental compose foundation UI into toolkit. Out of experimental in v1.6.6 (compose BOM 2024.08.00), can undo when we rev. * feedback * add keys to remember. fix concurrent modification of list. * remove print statement * remove print statement * remove unused typography style. feedback. * add new configurable styles. make theme internal. other feedback * comments * advanced options WIP * wip * simple color picker * advanced options * feedback * editable text: add done button to keyboard, clear focus on done. --------- Co-authored-by: Soren Roth <sor10874@esri.com> * trace config loading progress indicator * add UI strings and add utility networks strings.xml to t9nmanifest (#564) * UN Trace Tool: Adds navigation and AddStartingPoingScreen (#560) * UN trace tool compose test framework and cursory test case. (#561) * UN trace tool compose test framework and cursory test case. * change name of test to be more descriptive. remove unnecessary annotation. * use TraceState in tests, not anonymous object * remove unnecessary function --------- Co-authored-by: Soren Roth <sor10874@esri.com> * Sor10874/remove trace state iface (#567) * Changes to make identify work (#569) * Changes to make identify work * remove tappoint, screencoordinates as properties * optimize imports * Make the functions suspend * update padding * Adds Trace Result screen (#572) * Sor10874/real trace configs (#573) * Changes to make identify work * remove tappoint, screencoordinates as properties * save selected trace config in state obj * optimize imports * Make the functions suspend * wip * support real trace configs and real starting points * self review. formatting. visibility. * fix Symbology in composable PR suggestions * PR suggestions --------- Co-authored-by: Puneet Prakash <puneet_prakash@esri.com> Co-authored-by: Soren Roth <sor10874@esri.com> * change peek height so cancel starting point button is displayed (#578) Co-authored-by: Soren Roth <sor10874@esri.com> * use same instances in saver instantiation as well as restored version of DraggabbleAnchors (#579) Co-authored-by: Soren Roth <sor10874@esri.com> * Make sure trace works (#577) * WIP * WIP * WIP * WIP * WIP * update WIP * WIP * WIP * address code review comments * Trace state initialization (#574) * WIP * add functionality to initialize trace state * WIP * simplify app logic to initialize the viewmodel add optimization to the initialize method, to initialize only if state is not already initialized * WIP * Revert "simplify app logic to initialize the viewmodel" This reverts commit 3ed8557. * update formatting remove unused flow * WIP * initialized traceState in app viewmodel check if TraceState is already initialized * address code review comment * Update api file (#581) * Associate graphic with Starting points (#585) * WIP * associate graphic to each starting point * stable trace state (#583) * ensure Trace is skippable and restartable * encapsulate screen state to function * annotate TraceState * remove commented out code * remove unused parameter * get rid of extra initial navigation * dont reinitialize if already initializing or intiialized * PR feedback * remove compose compiler reporting --------- Co-authored-by: Soren Roth <sor10874@esri.com> * Make sure trace types work (#587) * WIP * WIP * remove check for empty traceResults * update doc * Changes to enable/disable Trace button (#588) * Changes to enable/disable Trace button * address code review comment * remove unused import * redesign trace options (#589) * redesign trace configuration selecter * ensure trace option spacing is uniform * starting points redesign wip * theme changes to support dark mode and light mode * remove unused code --------- Co-authored-by: Soren Roth <sor10874@esri.com> Co-authored-by: Puneet Prakash <puneet_prakash@esri.com> * Changes to make advanced options work (#591) * Changes to make advanced options work * refactor variable name * add doc * Update color selection * auto populate the tracename, persist zoom to results * address code review comments * update cancel button text to use string resource string (#596) * merge `l10n-staging` to `feature-branches/utilitynetworks` (#594) * Update pseudo translation - @mit10976 @Isa13169 @nia13404 (#570) * feat(localization): update pseudo translation (#584) * Update pseudo translation - @mit10976 @Isa13169 @nia13404 * Update translation - @mit10976 @Isa13169 @nia13404 --------- Co-authored-by: Jonathan Turpin <jturpin@esri.com> * StartingPoint detail with Slider port. (#592) * WIP StartingPoint detail with Slider port. * WIP * refactor slider code * update doc * rename StartingPointDetails -> StartingPointDetailsScreen * address code review comments * update package location * update tracetool unit test --------- Co-authored-by: Soren Roth <sor10874@esri.com> * Update slider functions to be internal (#599) * Add option to navigate to new trace from resultscreen (#597) * WIP * reset current trace * WIP * update padding values * remove newline * address code review comments * refactor tabrow * add newline * address code review comments * remove explicit name * remove unused import * Puneet/4525 multiple results (#601) * WIP * reset current trace * WIP * update padding values * remove newline * WIP * address code review comments * refactor tabrow * add newline * WIP * address code review comments * remove explicit name * revert changes to compass app * WIP * WIP * WIP * remove unused import * address code review comment * WIP * remove unused code * optimize imports * update currentTrace type * Utility Trace Tool: Add error reporting (#600) * revert change to clear starting points (#609) * Starting point details screen (#604) * WIP * WIP * add changes to support zoomto, delete and to select terminal * add newline * address code review comments * add padding * WIP * address Rama's comments * address Erick's comments. * few formatting updates * fix to have the icon always visible * Utility Trace Tool: Adds BackHandler actions (#606) * Make properties internal that don't need to be public (#615) * Puneet/4521 feature results (#608) * WIP * WIP * WIP * WIP * more changes * remove string * don't throw exceptions * add copyright * update formatting * fix to show multiple results * address code review comments * address comments * add backhandler * add string resource * update indentation * UN Trace Tool: Fix exceptions thrown when processing selected points (#614) * Result Screen - Advanced options (#613) * WIP * WIP * WIP * WIP * more changes * remove string * don't throw exceptions * add copyright * update formatting * WIP * WIP * WIP * WIP * fix to show multiple results * address code review comments * address comments * add backhandler * add string resource * WIP * update deletion logic * update logic * changes for color picker * refactor colorpicker * add newline * Save selected color in options screen * remove padding changes * revert changes to navigation * add copyright * update indentation * clear all the starting points * address code review comments * address code review comments * address code review feedback * Adds ExpandableCard State and uses it in TraceOptions (#619) * remove redundant navigation route (#625) * changes to disable the trace button while a trace is in progress (#617) * WIP * WIP * WIP * WIP * more changes * remove string * don't throw exceptions * add copyright * update formatting * WIP * WIP * WIP * WIP * fix to show multiple results * address code review comments * address comments * add backhandler * add string resource * WIP * update deletion logic * update logic * changes for color picker * refactor colorpicker * add newline * Save selected color in options screen * remove padding changes * revert changes to navigation * add copyright * update indentation * clear all the starting points * changes to disable the trace button while a trace is in progress * address code review comments * add progress indicator for add starting point screen * address code review comments * address code review feedback * remove duplicate function * address code review comment * address code review comments * remove task update * remove redundant setting of _isTaskInProgress * add content description strings to strings.xml (#623) * UN Trace Tool: Refactor TabRow (#622) * Show chevron only when traceResults size is greater than 1 (#632) * changes to add starting points as features (#621) * UN Trace Tool: UI Improvements (#626) * Fix for unchecking the TraceConfiguration in the expandable card once… (#637) * Fix for unchecking the TraceConfiguration in the expandable card once trace is done * Sort the trace configurations by name * fix for refreshing Advanced Options Name field everytime a new TraceConfiguration is selected (#636) * Update Result screen UI when the First element in the list is getting deleted (#628) * Update selected state for trace result graphics when 0th element gets deleted * update delete logic * address code review comment * UN Trace Tool: Applies modifier (#638) * Highlight feature results (#627) * WIP * add feature extent * WIP * some refactor * calculate geometries extent only if currentTraceExtent is null due to no feature extent * do memory intensive task on Default thread * group features only when list is not empty * Adds TraceScaffold composable (#641) * Add automated test for trace tool (#640) * UI test changes * WIP * add test * revert unintended changes * optimize imports * update formatting * Update Test * address code review comment * update doc * address code review comments * Merge v next (#645) * Upgrades to Kotlin 2.0 and K2 compiler and latest API build (#538) * Callout: Automated tests (#524) * L10n staging (#541) * Update pseudo translation - @mit10976 @Isa13169 @nia13404 (#510) * Update pseudo translation - @mit10976 @Isa13169 @nia13404 (#511) * Update translation - @mit10976 @Isa13169 @nia13404 (#536) --------- Co-authored-by: Jonathan Turpin <jturpin@esri.com> * Upgrade kotlin and compose compiler plugin version (#555) * Upgrade Kotlin version * update ksp version * Change Design of Dialog Authenticator Catalog (#547) * Added dialog change * Added dialog change * Added dialog change * Added AlertDialog to ServerTrust Challenge * Reflected feedback from reviews: Removed unneeded composable components, passed onDismiss instead of authenticatorState, moved some strings to strings.xml, left aligned all fields. * Reverted to old strings.xml, added padding, add UsernamePasswordAuthenticatorImpl, removed some parameters. * Fixed hostname, removed unnecessary column composables, fixed Authenticator comments, added back modifier to Authenticator * Fixed hostname, removed unnecessary column composables, fixed Authenticator comments, added back modifier to Authenticator * Added modifier back to ServerTrustAuthenticator and UsernamePasswordAuthenticatorImpl * Removed a modifier line, and added modifier comments in top level comments of API functions * Added modifier comment in top level, and passed modifier to AuthenticatorDelegate * Added review feedback, removed some comments, passed modifier to functions * Fixed mock hostname * Replaced url with hostname to be tested * Replaced all instances of original url to hostname for mock testing * Merge main - 200.6.0 (#568) * merge `Feature branches/forms` into v.next (#566) * `Forms`: Add `TextFormElement` (#542) * `Forms`: Add `TextFormElement` tests (#551) * add tests * updated feature form doc * bump sdk version * `Forms` : Add SubTypeFeatureLayer support (#559) * `Forms`: Fix stale `LaunchedEffect`s (#563) * fix stale launched effects * use rememberupdatedstate * update feature form doc (#565) * Remove unnecessary creating the viewmodel in the main activity (#571) * don't use trailing lambda to handle onClick in the Compass micro app (#618) * Merge Feature branches/forms into v.next (#611) * `Forms`: Add `TextFormElement` (#542) * `Forms`: Add `TextFormElement` tests (#551) * add tests * updated feature form doc * bump sdk version * `Forms` : Add SubTypeFeatureLayer support (#559) * `Forms`: Fix stale `LaunchedEffect`s (#563) * fix stale launched effects * use rememberupdatedstate * update feature form doc (#565) * show feature edit results related refactor (#575) * `Forms`: Show value for out of domain CodedValue types (#582) * update microapp api usage (#598) * `Forms` : Barcode support (#593) * `Forms`: Barcode Scanner Enhancements (#603) * `Forms` : Add Barcode input tests (#605) * update test maps (#610) * `Forms`: Barcode enhancements (#612) * add tap to focus * fix #apollo/914 * `Forms` : Update Doc (#616) * `Forms` : More Barcode Scanner Improvements (#620) * bump sdk version * PR feedback * Upgrade compose bom (#631) (#633) * upgrade compose bom to 2024.10.00 * remove unneeded test code --------- Co-authored-by: Erick Lopez Solis <erick_solis@esri.com> Co-authored-by: Shubham Sharma <shubhamsharma@esri.com> Co-authored-by: Jonathan Turpin <jturpin@esri.com> Co-authored-by: Puneet Prakash <puneet_prakash@esri.com> Co-authored-by: Nour Siwar @ Esri <100004210+NourdotSiwar@users.noreply.github.com> Co-authored-by: Kaushik Meesala <kaushik.pulagara@gmail.com> Co-authored-by: Gunther Heppner <gheppner@esri.com> Co-authored-by: Kaushik Meesala <kmeesala@esri.com> Co-authored-by: Soren Roth <sor10874@esri.com> * Merge v next into feature branch (#650) * Upgrades to Kotlin 2.0 and K2 compiler and latest API build (#538) * Callout: Automated tests (#524) * L10n staging (#541) * Update pseudo translation - @mit10976 @Isa13169 @nia13404 (#510) * Update pseudo translation - @mit10976 @Isa13169 @nia13404 (#511) * Update translation - @mit10976 @Isa13169 @nia13404 (#536) --------- Co-authored-by: Jonathan Turpin <jturpin@esri.com> * Upgrade kotlin and compose compiler plugin version (#555) * Upgrade Kotlin version * update ksp version * Change Design of Dialog Authenticator Catalog (#547) * Added dialog change * Added dialog change * Added dialog change * Added AlertDialog to ServerTrust Challenge * Reflected feedback from reviews: Removed unneeded composable components, passed onDismiss instead of authenticatorState, moved some strings to strings.xml, left aligned all fields. * Reverted to old strings.xml, added padding, add UsernamePasswordAuthenticatorImpl, removed some parameters. * Fixed hostname, removed unnecessary column composables, fixed Authenticator comments, added back modifier to Authenticator * Fixed hostname, removed unnecessary column composables, fixed Authenticator comments, added back modifier to Authenticator * Added modifier back to ServerTrustAuthenticator and UsernamePasswordAuthenticatorImpl * Removed a modifier line, and added modifier comments in top level comments of API functions * Added modifier comment in top level, and passed modifier to AuthenticatorDelegate * Added review feedback, removed some comments, passed modifier to functions * Fixed mock hostname * Replaced url with hostname to be tested * Replaced all instances of original url to hostname for mock testing * Merge main - 200.6.0 (#568) * merge `Feature branches/forms` into v.next (#566) * `Forms`: Add `TextFormElement` (#542) * `Forms`: Add `TextFormElement` tests (#551) * add tests * updated feature form doc * bump sdk version * `Forms` : Add SubTypeFeatureLayer support (#559) * `Forms`: Fix stale `LaunchedEffect`s (#563) * fix stale launched effects * use rememberupdatedstate * update feature form doc (#565) * Remove unnecessary creating the viewmodel in the main activity (#571) * don't use trailing lambda to handle onClick in the Compass micro app (#618) * Merge Feature branches/forms into v.next (#611) * `Forms`: Add `TextFormElement` (#542) * `Forms`: Add `TextFormElement` tests (#551) * add tests * updated feature form doc * bump sdk version * `Forms` : Add SubTypeFeatureLayer support (#559) * `Forms`: Fix stale `LaunchedEffect`s (#563) * fix stale launched effects * use rememberupdatedstate * update feature form doc (#565) * show feature edit results related refactor (#575) * `Forms`: Show value for out of domain CodedValue types (#582) * update microapp api usage (#598) * `Forms` : Barcode support (#593) * `Forms`: Barcode Scanner Enhancements (#603) * `Forms` : Add Barcode input tests (#605) * update test maps (#610) * `Forms`: Barcode enhancements (#612) * add tap to focus * fix #apollo/914 * `Forms` : Update Doc (#616) * `Forms` : More Barcode Scanner Improvements (#620) * bump sdk version * PR feedback * Upgrade compose bom (#631) (#633) * upgrade compose bom to 2024.10.00 * remove unneeded test code --------- Co-authored-by: Erick Lopez Solis <erick_solis@esri.com> Co-authored-by: Shubham Sharma <shubhamsharma@esri.com> Co-authored-by: Jonathan Turpin <jturpin@esri.com> Co-authored-by: Puneet Prakash <puneet_prakash@esri.com> Co-authored-by: Nour Siwar @ Esri <100004210+NourdotSiwar@users.noreply.github.com> Co-authored-by: Kaushik Meesala <kaushik.pulagara@gmail.com> Co-authored-by: Gunther Heppner <gheppner@esri.com> Co-authored-by: Kaushik Meesala <kmeesala@esri.com> Co-authored-by: Soren Roth <sor10874@esri.com> * Utility Network update api file (#643) * update api file * add BuildConfig to api file ignores --------- Co-authored-by: Soren Roth <sor10874@esri.com> --------- Co-authored-by: Soren Roth <sor10874@esri.com> Co-authored-by: Erick Lopez Solis <erick_solis@esri.com> Co-authored-by: Puneet Prakash <puneet_prakash@esri.com> Co-authored-by: Jonathan Turpin <jturpin@esri.com> Co-authored-by: Shubham Sharma <shubhamsharma@esri.com> Co-authored-by: Nour Siwar @ Esri <100004210+NourdotSiwar@users.noreply.github.com> Co-authored-by: Kaushik Meesala <kaushik.pulagara@gmail.com> Co-authored-by: Gunther Heppner <gheppner@esri.com> Co-authored-by: Kaushik Meesala <kmeesala@esri.com>

puneet-pdx added 2 commits September 30, 2024 11:47

WIP

c54d61a

reset current trace

620fec3

puneet-pdx self-assigned this Sep 30, 2024

puneet-pdx added 3 commits September 30, 2024 15:23

WIP

effe5f9

update padding values

02d9e3a

remove newline

14b8f76

sorenoid self-requested a review October 2, 2024 20:12

sorenoid requested changes Oct 2, 2024

View reviewed changes

address code review comments

8ef418b

puneet-pdx requested a review from sorenoid October 3, 2024 00:50

puneet-pdx added 2 commits October 3, 2024 06:36

refactor tabrow

048642b

add newline

303fbed

sorenoid requested changes Oct 4, 2024

View reviewed changes

address code review comments

a0d4df8

puneet-pdx requested a review from sorenoid October 4, 2024 18:05

sorenoid approved these changes Oct 4, 2024

View reviewed changes

remove explicit name

069f389

eri9000 self-requested a review October 7, 2024 19:57

eri9000 approved these changes Oct 7, 2024

View reviewed changes

remove unused import

094570c

puneet-pdx merged commit 1bc2f18 into feature-branches/utilitynetworktrace Oct 7, 2024

puneet-pdx deleted the puneet/4492_NavigateToNewTrace branch October 7, 2024 23:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add option to navigate to new trace from resultscreen #597

Add option to navigate to new trace from resultscreen #597

puneet-pdx commented Sep 30, 2024 •

edited

Loading

sorenoid left a comment

sorenoid Oct 2, 2024

sorenoid Oct 2, 2024

sorenoid Oct 2, 2024

sorenoid Oct 2, 2024

sorenoid Oct 2, 2024

puneet-pdx Oct 3, 2024

sorenoid Oct 2, 2024

sorenoid Oct 2, 2024

sorenoid left a comment

sorenoid Oct 4, 2024

puneet-pdx Oct 4, 2024

sorenoid Oct 4, 2024

puneet-pdx Oct 4, 2024

sorenoid left a comment

sorenoid Oct 4, 2024

sorenoid Oct 4, 2024

eri9000 left a comment

eri9000 Oct 7, 2024

eri9000 Oct 7, 2024

puneet-pdx Oct 7, 2024

	var selectedTabIndex by remember { mutableIntStateOf(0) }
	var selectedTabIndex by rememberSaveable { mutableIntStateOf(0) }

	TabRow(onBackToResults, selectedIndex = 0)
	TabRow(onBackToResults, 0)

	TabRow(onBackToNewTrace, selectedIndex = 1)
	TabRow(onBackToNewTrace, 1)

Add option to navigate to new trace from resultscreen #597

Add option to navigate to new trace from resultscreen #597

Conversation

puneet-pdx commented Sep 30, 2024 • edited Loading

Description:

Summary of changes:

Pre-merge Checklist

sorenoid left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sorenoid left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sorenoid left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

eri9000 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

puneet-pdx commented Sep 30, 2024 •

edited

Loading