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 links and text to the template and tutorial #161

Merged
merged 3 commits into from
Aug 29, 2023

Conversation

MongoCaleb
Copy link
Collaborator

Copy link
Collaborator

@dacharyc dacharyc left a comment

Choose a reason for hiding this comment

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

Back to you with some suggestions and copy-pasta fixes!


## Configuration

Ensure `flutter_todo/assets/config/atlasConfig.json` exists and contains the following properties:
For this template app to work, you must ensure that `flutter_todo/assets/config/atlasConfig.json` exists and contains the following properties:
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a nitpick for some of these, but presumably the people who are consuming this README will be doing so either from the artifact repo (https://github.com/mongodb/template-app-dart-flutter-todo) or as a download from the Atlas UI where flutter_todo is not part of the directory structure. So I think when we're providing the path, we should do it relative to where the README is located and not make any assumptions about the enclosing directory structure.

Suggested change
For this template app to work, you must ensure that `flutter_todo/assets/config/atlasConfig.json` exists and contains the following properties:
For this template app to work, you must ensure that `assets/config/atlasConfig.json` exists and contains the following properties:

(Also, if you take this suggestion, we should change it in ln 39 down below, too, I just can't leave a comment there since we're not modifying it here.)

## Configuration

Ensure `app/src/main/res/values/atlasConfig.xml` exists and contains the following properties:
For this template app to work, you must ensure that `kotlin-sdk/app/src/main/res/values/atlasConfig.xml` exists and contains the following properties:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same note as above re: the directory structure.

Suggested change
For this template app to work, you must ensure that `kotlin-sdk/app/src/main/res/values/atlasConfig.xml` exists and contains the following properties:
For this template app to work, you must ensure that `app/src/main/res/values/atlasConfig.xml` exists and contains the following properties:


The easiest way to use this template app is to log on to [Atlas App Services](https://realm.mongodb.com/) and click the **Create App From Template** button. Choose
**Real Time Sync**, and then follow the prompts. While the backend app is being
created, you can download this Flutter template app pre-configured for your new
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
created, you can download this Flutter template app pre-configured for your new
created, you can download this React Native template app pre-configured for your new


The easiest way to use this template app is to log on to [Atlas App Services](https://realm.mongodb.com/) and click the **Create App From Template** button. Choose
**Real Time Sync**, and then follow the prompts. While the backend app is being
created, you can download this Flutter template app pre-configured for your new
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
created, you can download this Flutter template app pre-configured for your new
created, you can download this Maui template app pre-configured for your new

- **realm_base_url** the App Services backend URL. This should be https://realm.mongodb.com in most cases.
The easiest way to use this template app is to log on to [Atlas App Services](https://realm.mongodb.com/) and click the **Create App From Template** button. Choose
**Real Time Sync**, and then follow the prompts. While the backend app is being
created, you can download this Flutter template app pre-configured for your new
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
created, you can download this Flutter template app pre-configured for your new
created, you can download this Kotlin template app pre-configured for your new

@@ -1,5 +1,10 @@
# Maui C# Template App

A todo list application built with the [.NET Flutter SDK](https://www.mongodb.com/docs/realm/sdk/dotnet/) and [Atlas Device Sync](https://www.mongodb.com/docs/atlas/app-services/sync/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
A todo list application built with the [.NET Flutter SDK](https://www.mongodb.com/docs/realm/sdk/dotnet/) and [Atlas Device Sync](https://www.mongodb.com/docs/atlas/app-services/sync/).
A todo list application built with the [.NET SDK](https://www.mongodb.com/docs/realm/sdk/dotnet/) and [Atlas Device Sync](https://www.mongodb.com/docs/atlas/app-services/sync/).

@@ -1,5 +1,10 @@
# Maui C# Template App

A todo list application built with the [.NET Flutter SDK](https://www.mongodb.com/docs/realm/sdk/dotnet/) and [Atlas Device Sync](https://www.mongodb.com/docs/atlas/app-services/sync/).

You can follow along with the [Flutter Tutorial](https://www.mongodb.com/docs/atlas/app-services/tutorial/dotnet/) to see how to build, modify, and
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
You can follow along with the [Flutter Tutorial](https://www.mongodb.com/docs/atlas/app-services/tutorial/dotnet/) to see how to build, modify, and
You can follow along with the [Maui Tutorial](https://www.mongodb.com/docs/atlas/app-services/tutorial/dotnet/) to see how to build, modify, and

@@ -1,14 +1,26 @@
# SwiftUI Template App

A todo list application built with the [SwiftUI SDK](https://www.mongodb.com/docs/realm/sdk/swift/) and [Atlas Device Sync](https://www.mongodb.com/docs/atlas/app-services/sync/).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
A todo list application built with the [SwiftUI SDK](https://www.mongodb.com/docs/realm/sdk/swift/) and [Atlas Device Sync](https://www.mongodb.com/docs/atlas/app-services/sync/).
A todo list application built with the [Realm Swift SDK](https://www.mongodb.com/docs/realm/sdk/swift/) and [Atlas Device Sync](https://www.mongodb.com/docs/atlas/app-services/sync/).


The easiest way to use this template app is to log on to [Atlas App Services](https://realm.mongodb.com/) and click the **Create App From Template** button. Choose
**Real Time Sync**, and then follow the prompts. While the backend app is being
created, you can download this Flutter template app pre-configured for your new
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
created, you can download this Flutter template app pre-configured for your new
created, you can download this SwiftUI template app pre-configured for your new

Copy link
Collaborator

@dacharyc dacharyc left a comment

Choose a reason for hiding this comment

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

Teeny tiny nit but LGTM!

sync-todo/v2/client/kotlin-sdk/README.md Outdated Show resolved Hide resolved
@MongoCaleb MongoCaleb merged commit 1d2f139 into mongodb-university:main Aug 29, 2023
1 check passed
@MongoCaleb MongoCaleb deleted the 32473 branch August 29, 2023 15:53
@MongoCaleb MongoCaleb restored the 32473 branch August 29, 2023 15:56
@MongoCaleb MongoCaleb deleted the 32473 branch August 29, 2023 16:14
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.

2 participants