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

Spelling #2193

Merged
merged 12 commits into from
Oct 3, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ To contribute to the project, please see [Contributing](https://github.com/googl

You can create a [GitHub issue](https://github.com/google/android-fhir/issues) for bugs and feature requests.

In case you find any security bug, please do NOT create a Github issue. Instead, email us at <[email protected]>.
In case you find any security bug, please do NOT create a GitHub issue. Instead, email us at <[email protected]>.
vorburger marked this conversation as resolved.
Show resolved Hide resolved

If you want to provide any feedback or discuss use cases you can:
* email us at <[email protected]>
Expand Down
2 changes: 1 addition & 1 deletion catalog/src/main/assets/component_open_choice.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
}
}
],
"text": "Do you have any pre-existing health conditions?",
"text": "Do you have any preexisting health conditions?",
"item": [
{
"linkId": "1.1",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
}
],
"linkId": "1",
"text": "Do you have any pre-existing health conditions?",
"text": "Do you have any preexisting health conditions?",
"required": true,
"item": [
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -845,7 +845,7 @@ fun Questionnaire.QuestionnaireItemComponent.createQuestionnaireResponseItem():
}

/**
* Returns a list of answers from the initial values of the questionnaire item. `null` if no intial
* Returns a list of answers from the initial values of the questionnaire item. `null` if no initial
* value.
*/
private fun Questionnaire.QuestionnaireItemComponent.createQuestionnaireResponseItemAnswers():
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1078,7 +1078,7 @@ class ResourceMapperTest {
}

@Test
fun `extract_updateIntegerObservationForDecimalDefination_shouldUpdateAsDecimal() `() =
fun `extract_updateIntegerObservationForDecimalDefinition_shouldUpdateAsDecimal() `() =
runBlocking {
@Language("JSON")
val questionnaireJson =
Expand Down
4 changes: 2 additions & 2 deletions docs/data-capture/scripts/navigation-loader.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ revealParents = (part) => {
};

/*
This is a work-around for safari being IE of our times.
It doesn't fire a DOMContentLoaded, presumabely because eventListener is added after it wants to do it
This is a workaround for safari being IE of our times.
It doesn't fire a DOMContentLoaded, presumably because eventListener is added after it wants to do it
*/
if (document.readyState == 'loading') {
window.addEventListener('DOMContentLoaded', () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ <h1 class="cover"><span>sync</span><wbr></wbr><span>Constraints</span></h1>
<div>
<div class="platform-hinted " data-platform-hinted="data-platform-hinted"><div class="content sourceset-depenent-content" data-active="" data-togglable=":engine:dokkaHtml/release"><div class="symbol monospace">val <a href="sync-constraints.html">syncConstraints</a>: <a href="https://developer.android.com/reference/kotlin/androidx/work/Constraints.html">Constraints</a><span class="top-right-position"><span class="copy-icon"></span><div class="copy-popup-wrapper popup-to-left"><span class="copy-popup-icon"></span><span>Content copied to clipboard</span></div></span></div></div></div>
</div>
<p class="paragraph">Constraints that specify the requirements needed before the synchronisation is triggered. E.g. network type (Wifi, 3G etc), the device should be charging etc.</p></div>
<p class="paragraph">Constraints that specify the requirements needed before the synchronisation is triggered. E.g. network type (WiFi, 3G etc), the device should be charging etc.</p></div>
<h2 class="">Sources</h2>
<div class="table" data-togglable="Sources"><a data-name="-1831107255%2FSource%2F-2001932064" anchor-label="https://github.com/google/android-fhir/tree/master/engine/src/main/java/com/google/android/fhir/sync/Config.kt#L51" id="-1831107255%2FSource%2F-2001932064" data-filterable-set=":engine:dokkaHtml/main"></a>
<div class="table-row" data-filterable-current=":engine:dokkaHtml/main" data-filterable-set=":engine:dokkaHtml/main">
Expand Down
4 changes: 2 additions & 2 deletions docs/engine/scripts/navigation-loader.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ revealParents = (part) => {
};

/*
This is a work-around for safari being IE of our times.
It doesn't fire a DOMContentLoaded, presumabely because eventListener is added after it wants to do it
This is a workaround for safari being IE of our times.
It doesn't fire a DOMContentLoaded, presumably because eventListener is added after it wants to do it
*/
if (document.readyState == 'loading') {
window.addEventListener('DOMContentLoaded', () => {
Expand Down
4 changes: 2 additions & 2 deletions docs/workflow/scripts/navigation-loader.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ revealParents = (part) => {
};

/*
This is a work-around for safari being IE of our times.
It doesn't fire a DOMContentLoaded, presumabely because eventListener is added after it wants to do it
This is a workaround for safari being IE of our times.
It doesn't fire a DOMContentLoaded, presumably because eventListener is added after it wants to do it
*/
if (document.readyState == 'loading') {
window.addEventListener('DOMContentLoaded', () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ object SyncDataParams {
class PeriodicSyncConfiguration(
/**
* Constraints that specify the requirements needed before the synchronisation is triggered. E.g.
* network type (Wifi, 3G etc), the device should be charging etc.
* network type (WiFi, 3G etc), the device should be charging etc.
*/
val syncConstraints: Constraints = Constraints.Builder().build(),

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ internal constructor(
}
}

/** Imports the Knolwedge Artifact from the provided [file] to the default dependency. */
/** Imports the Knowledge Artifact from the provided [file] to the default dependency. */
suspend fun install(file: File) {
importFile(null, file)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ object CqlBuilder : Loadable() {
}

/**
* Assembles an ELM Library exported as a JSON in to a FHIRLibrary
* Assembles an ELM Library exported as a JSON into a FHIRLibrary
vorburger marked this conversation as resolved.
Show resolved Hide resolved
*
* @param jsonElmStr the JSON representation of the ELM Library
* @param libName the Library name
Expand Down Expand Up @@ -218,7 +218,7 @@ object CqlBuilder : Loadable() {
}

fun generatesFhirLibraryEqualsTo(expectedFhirAssetName: String): CompiledCql {
// Given the ELM is the same, builds the lib with the expented, not the new ELM to make sure
// Given the ELM is the same, builds the lib with the expected, not the new ELM to make sure
// the base 64 representation of the Library matches.
val library =
assembleFhirLib(
Expand Down