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

Add info grey box in calculated expression behavior screen in catalog app #1679

Conversation

PallaviGanorkar
Copy link
Contributor

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).

Fixes #1525

Description
feature in ticket #1525 was partially implemented as part of PR #1380.

Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?

Type
Choose one: Feature

Screenshots (if applicable)

Checklist

  • I have read and acknowledged the Code of conduct.
  • I have read the Contributing page.
  • I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
  • I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
  • I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
  • I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
  • I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

@PallaviGanorkar PallaviGanorkar force-pushed the pg/1525_greybox_for_info_in_calculated_expression branch from 38319c5 to 3100dce Compare October 31, 2022 10:00
@jingtang10 jingtang10 changed the title Catlog app - Info grey box added in calculated expression behavior screen Add info grey box in calculated expression behavior screen in catalog app Nov 2, 2022
Copy link
Collaborator

@jingtang10 jingtang10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jingtang10
Copy link
Collaborator

please add a screenshot before merging

@PallaviGanorkar
Copy link
Contributor Author

please add a screenshot before merging

calculated_expression_behaviour

@PallaviGanorkar PallaviGanorkar merged commit 4e88f68 into google:master Nov 3, 2022
ktarasenko pushed a commit to ktarasenko/android-fhir that referenced this pull request Nov 7, 2022
ktarasenko pushed a commit to ktarasenko/android-fhir that referenced this pull request Nov 14, 2022
ktarasenko added a commit that referenced this pull request Nov 23, 2022
* Add Implementation guide management library module

* Crashed Fixed in Quantity Input Widget (#1673)

* Crashed Fixed in Quantity Input Widget

* reverted unwanted changes

* Spotless check applied

* remove clear answer from factory

Co-authored-by: omarismail <[email protected]>

* Fix review comments

* Updated code to set the radio button in a radio group using radio groups check api instead of radio button apis (#1691)

* Bump up SDC version to beta06 (#1694)

* Follow up code health fixes of PR 1442 (#1631)

* WIP PR feedback

* spotless ran

* added test cases for validation util

* used fhirpathengine instance from fhir path expression evaluator

* updated test cases name and convention

* updated test cases name and their convention to look consistent

Co-authored-by: Jing Tang <[email protected]>

* Disable leniency in DateFormat lib when converting date (#1687)

* Fix Issue #1638: disable lenient check

* add espresso test

* add more tests per date field changed

* Regenerate docs after SDC version bump (#1696)

* Documentation Update : Changed the folder structure and did some other minor add-ons (#1662)

* Create CODE_OF_CONDUCT.md

* Delete CODE_OF_CONDUCT.md

* Create Contributing.md

* Delete contributing.md

* Update README.md

* Update README.md

Co-authored-by: Omar Ismail <[email protected]>

* Catlog app- Info grey box added in Calculated expression screen Behavior (#1679)

* Scroll down the recyclerview to show the Add answer button. (#1666)

* Scroll down the recyclerview to show the Add answer button.

* Address review comments.

* Address review comments.

* Add flaky flag to retry

* update gcloud components as well

* remove version flag

Co-authored-by: Santosh Pingle <[email protected]>
Co-authored-by: Omar Ismail <[email protected]>
Co-authored-by: omarismail <[email protected]>

* Enable "Next button" on paginated view when nested answer is chosen (#1684)

Fix Issue  #1677

* Add trailing line break.

* Update buildSrc/src/main/kotlin/Releases.kt

Co-authored-by: Jing Tang <[email protected]>

Co-authored-by: PallaviGanorkar <[email protected]>
Co-authored-by: omarismail <[email protected]>
Co-authored-by: aditya-07 <[email protected]>
Co-authored-by: Omar Ismail <[email protected]>
Co-authored-by: aurangzaibumer <[email protected]>
Co-authored-by: Jing Tang <[email protected]>
Co-authored-by: Vanshaj Poonia <[email protected]>
Co-authored-by: santosh-pingle <[email protected]>
Co-authored-by: Santosh Pingle <[email protected]>
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.

SDC catalog app: show calculation behavior
3 participants