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

Lauren fp ts exercises #151

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Lauren fp ts exercises #151

wants to merge 16 commits into from

Conversation

lauren-inato
Copy link
Contributor

FYI: I'm keeping this as a draft PR so that I don't mistakenly merge it!

@lauren-inato lauren-inato requested a review from LaureRC July 25, 2023 14:41
)

// I checked the solution for the above exercise because I was unable to figure out
// that "rte.fromReader" needed to proceed the code accessing the TimeService dependency.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's discuss this in a call then in case you want some extra explanations about this?

@@ -178,10 +220,25 @@ export const pageViewsB = new Map(
//
// In case a page appears in both sources, their view count should be summed.

export const allPageViews: ReadonlyMap<string, Analytics> = unimplemented();

////// I needed to look at the solution for this part. I don't feel like I had the tools to figure this out on my own.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let me know if you want to discuss this live!

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