Skip to content

Taimur_COVID19_Week1 #40

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Taimur_COVID19_Week1 #40

wants to merge 4 commits into from

Conversation

taimurkashif
Copy link

@taimurkashif taimurkashif commented Apr 13, 2020 •
edited
Loading

Issues Closed

Closes #2

Closes #3
...

Changes proposed in this pull request:

  • Presentation was completed

@kavuong

@taimurkashif taimurkashif requested a review from kavuong April 13, 2020 17:23
@vercel
Copy link

vercel bot commented Apr 13, 2020 •
edited
Loading

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/bitproject/mdx-deck/5ohmgs3cf
✅ Preview: https://mdx-deck-git-taimur.bitproject.now.sh

kavuong
kavuong suggested changes Apr 14, 2020
View reviewed changes
Copy link
Contributor

@kavuong kavuong left a comment

Choose a reason for hiding this comment

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

@taimurkashif can you remove all of the cache and installation files?

@taimurkashif
Revert "commiting many files"
6edc49e 
This reverts commit 73b610c.
@taimurkashif
Updated COVID19.mdx
ad9d68e 
Updated COVID19.mdx to the file which Joey and Jeffrey updated. It now loads in browser, with speaker notes
@taimurkashif
Copy link
Author

taimurkashif commented Apr 14, 2020 •
edited
Loading

@kavuong extra files removed! Also note that the new presentation is one that @joeyhuaa and @JeffTheAggie fixed and it is now loading in browser.

@kavuong kavuong self-requested a review April 16, 2020 06:47
kavuong
kavuong suggested changes Apr 16, 2020
View reviewed changes
Copy link
Contributor

@kavuong kavuong left a comment

Choose a reason for hiding this comment

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

Please make sure your .mdx file is in the proper place and that all of the newly created assets by @joeyhuaa and @JeffTheAggie are in the right places.

Also please set up a Code Sandbox link for your branch per the guide here: https://about.bitproject.org/teams/developer-relations/setting-up-code-sandbox#for-managers-incoming-managers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kavuong kavuong kavuong requested changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Backend] [COVID19] Week 1 of 3: Manager Tasks [Backend] [COVID19] Week 1 of 3: Dev Tasks

2 participants

@taimurkashif @kavuong