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

AC-746 NPE removed at Visit Dashboard #721

Merged
merged 1 commit into from
Apr 22, 2020
Merged

Conversation

rishabh-997
Copy link
Collaborator

Description of what I changed

The error was because of a non-null variable in resource file which was being assigned null from the server, so changed it to accumulate null value.

Issue I worked on

JIRA Issue: https://issues.openmrs.org/browse/AC-746

Checklist: I completed these to help reviewers :)

  • My pull request only contains ONE single commit
    (the number above, next to the 'Commits' tab is 1).
  • I have added tests to cover my changes. (If you refactored
    existing code that was well tested you do not have to add tests)
  • All new and existing tests passed.
  • My pull request is based on the latest changes of the master branch.

@@ -554,7 +536,7 @@
"views": [],
"setupQueries": [
"CREATE TABLE IF NOT EXISTS room_master_table (id INTEGER PRIMARY KEY,identity_hash TEXT)",
"INSERT OR REPLACE INTO room_master_table (id,identity_hash) VALUES(42, \"f2ba83e78fa9d0190a4a24b5be11b2de\")"
"INSERT OR REPLACE INTO room_master_table (id,identity_hash) VALUES(42, \"41a50b7fab10cb07ddedc8846a9b55f5\")"
]
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

@rishabh-997 this file pops up with its modified version always when we build but I think you should keep this file out of this pull request as it is not directly related to this Issue !!

@codecov-io
Copy link

codecov-io commented Apr 13, 2020

Codecov Report

Merging #721 into master will not change coverage by %.
The diff coverage is 50.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #721   +/-   ##
=======================================
  Coverage   11.90%   11.90%           
=======================================
  Files         236      236           
  Lines       10502    10502           
  Branches     1006     1006           
=======================================
  Hits         1250     1250           
  Misses       9171     9171           
  Partials       81       81           
Impacted Files Coverage Δ
...main/java/org/openmrs/mobile/models/Observation.kt 23.07% <0.00%> (ø)
...rc/main/java/org/openmrs/mobile/models/Resource.kt 44.44% <100.00%> (ø)
...c/main/java/org/openmrs/mobile/models/VisitType.kt 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 114b011...7bafa76. Read the comment docs.

removed changes in 1.json

NPE removed at visit dashboard
Copy link
Collaborator

@f4ww4z f4ww4z left a comment

Choose a reason for hiding this comment

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

@f4ww4z f4ww4z merged commit 88b9ae7 into openmrs:master Apr 22, 2020
@rishabh-997 rishabh-997 deleted the AC-746 branch May 12, 2020 10:30
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.

4 participants