data-umbrella / event-transcripts

transcripts from our recorded events
https://www.youtube.com/c/dataumbrella/videos
36 stars 35 forks source link

Add timestamps: video 61b (React Part 2) #155

Closed nicholeboaz closed 1 year ago

nicholeboaz commented 1 year ago

[61b] React Tutorial Part 2: JavaScript, Node.js

Towards #92

welcome[bot] commented 1 year ago

Thank You Banner :sparkling_heart: Thanks for opening this pull request! :sparkling_heart: Data Umbrella community really appreciates your time and effort to contribute to the project. Please make sure you have read our Contributing Guidelines and filled in our pull request template to the best of your ability.

reshamas commented 1 year ago

@nicholeboaz Thank you for this PR! Very nice job with the timestamps, and thank you for reading the instructions on submitting a PR!

Would you like to accept / push the suggested changes, and then I will merge?

reshamas commented 1 year ago

@nicholeboaz Also, can you sync your repo? Someone recently submitted timestamps for the React video Part 1.

nicholeboaz commented 1 year ago

I think I synced my repo but it's possible I didn't do it correctly -- please let me know if there's anything else I need to do.

reshamas commented 1 year ago

@nicholeboaz Do you want to start a with a fresh PR in a new branch, so you can get credit for your commits? This is currently what I see, which means the [61a] timestamps aren't showing, so it isn't synced. Starting with a new branch might be the fastest way to solve it.

## Timestamps
[61a] React Tutorial Part 1: Background (Joe Torreggiani)   

## Agenda
[61b] React Tutorial Part 2: JavaScript, Node.js (Joe Torreggiani)   

## Agenda
05:22 What makes JavaScript interpreted?
06:10 Object-oriented, imperative, and functional programming in JavaScript
07:41 Q&A: Are there efficiency or speed differences betw
nicholeboaz commented 1 year ago

Yes, I can make a new PR! However, I'm not seeing the 61a timestamps on the most up-to-date file on the main branch here, is it possible I'm looking in the wrong place?

reshamas commented 1 year ago

@nicholeboaz I had merged the Part 1 timestamps in and somehow they disappeared, probably a merge overwrite.

I added them back in: https://github.com/data-umbrella/event-transcripts/blob/main/2022/61-joe-react.md

Can you now: a) sync your repo b) start a new branch c) add in part 2 timestamps