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 ECON1101 #106

Merged
merged 19 commits into from
Jan 30, 2024
Merged

add ECON1101 #106

merged 19 commits into from
Jan 30, 2024

Conversation

TLNA114514
Copy link

Add a new course

@why-in-Shanghaitech
Copy link
Member

Hi! Thank you very much for your contribution! I'll review this PR in a few days.
I've noticed that this is your first contribution. I'll add you to the contributor list once the pull request is accepted.

Copy link
Member

@why-in-Shanghaitech why-in-Shanghaitech left a comment

Choose a reason for hiding this comment

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

Waiting for author response...

courses/ECON1101/meta.json Show resolved Hide resolved
"hours": "48",
"prerequisite": "无",
"evaluation": "20% homework + 15% quiz + 15% midterm + 50% final",
"teacher": [],

Choose a reason for hiding this comment

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

I've noticed that the instructor is Xiyi Yang and the TA is Dong Zhang. Is this correct? If you could confirm this info, I could fix the teacher and ta lists for you.

@TLNA114514
Copy link
Author

TLNA114514 commented Jan 29, 2024 via email

@TLNA114514
Copy link
Author

TLNA114514 commented Jan 29, 2024 via email

Copy link
Member

@why-in-Shanghaitech why-in-Shanghaitech left a comment

Choose a reason for hiding this comment

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

Thank you so much for your quick reply. I'll merge this PR after fixing the teacher and ta lists.

why-in-Shanghaitech added a commit that referenced this pull request Jan 30, 2024
@why-in-Shanghaitech why-in-Shanghaitech merged commit ddeb63e into i-TechX:file-base Jan 30, 2024
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.

None yet

2 participants