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

Update README #11

Merged
merged 4 commits into from
Mar 10, 2024
Merged

Update README #11

merged 4 commits into from
Mar 10, 2024

Conversation

tanjieling0
Copy link

No description provided.

Copy link

@starsia starsia left a comment

Choose a reason for hiding this comment

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

Minor typos, otherwise LGTM 🤩

tanjieling0 and others added 2 commits March 8, 2024 02:16
Co-authored-by: Zhong Qing <[email protected]>
Co-authored-by: Zhong Qing <[email protected]>
@starsia starsia merged commit a73907f into AY2324S2-CS2103T-F12-1:master Mar 10, 2024
3 checks passed
@tanjieling0
Copy link
Author

fixes #10

@tanjieling0 tanjieling0 self-assigned this Mar 11, 2024
@tanjieling0 tanjieling0 added this to the V1.1 milestone Mar 11, 2024
Copy link
Collaborator

@shaunlxw shaunlxw left a comment

Choose a reason for hiding this comment

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

LGTM less the minor changes! we are required to acknowledge the original source of the code (https://se-education.org). i'm not sure if the last line in README.md is sufficient as acknowledgement, or if it'll even be there after we change it to fit our project.

@@ -1,14 +1,35 @@
[![CI Status](https://github.com/se-edu/addressbook-level3/workflows/Java%20CI/badge.svg)](https://github.com/se-edu/addressbook-level3/actions)
Copy link
Collaborator

Choose a reason for hiding this comment

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

need to change this by generating our own workflow badge. not sure if it's only for Java CI only or for all workflow though..

To begin your journey with NetConnect, simply download the latest release from [here]() and refer to the [User Guide](docs/UserGuide.md)
for detailed guidance in using the application.

For the detailed documentation of this project, see the **[Address Book Product Website](https://se-education.org/addressbook-level3)**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

i think we need to change the description and link of the documentation to ours

Copy link
Collaborator

Choose a reason for hiding this comment

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

our group needs to decide whether to use the term "supplier" or "partner"

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.

3 participants