Skip to content

London | 25-ITP-Sept | Samuel Tarawally | Sprint 1 | Coursework #819

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 15 commits into
base: main
Choose a base branch
from
Open

London | 25-ITP-Sept | Samuel Tarawally | Sprint 1 | Coursework #819

wants to merge 15 commits into from

Conversation

@Tarawally
Copy link

@Tarawally Tarawally commented Oct 29, 2025 •
edited
Loading

Learners, PR Template

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Changelist

  • key exercises: Variable reassignment, string extraction, file path parsing, random number generation
  • mandatory errors: Fixed comment syntax, const reassignment, temporal dead zone, type errors, invalid variable names
  • mandatory interpret: String-to-number conversion, time formatting, currency conversion

@Tarawally
docs(key-exercises): explain variable reassignment in 1-count.js
9b83b30 
Describe how the assignment operator evaluates the right-hand side
expression before assigning the result to the count variable
@Tarawally
feat: add initials variable from first, middle, and last names
b6ec84d 
Concatenate first characters of firstName, middleName, and lastName
to create initials string dynamically
@Tarawally
feat(key-exercies): complete paths by deriving dir and ext
a3978a4
@Tarawally
feat: add detailed breakdown of random number generation logic
8e8c891
@Tarawally
chore(gitignore): broaden ignores for Node/JS, IDEs, env and docs
09dc15f
@Tarawally
refactor(sprint-1): simplify initials template literal
103635d
@Tarawally
fix(sprint-1): resolve ReferenceError by declaring greeting outside b…
d7626de 
…lock scope
@Tarawally
fix(sprint-1): resolve TypeError by using let instead of const
78c9889
@Tarawally
fix(sprint-1): resolve SyntaxError by using unique variable names
4d64b5e
@Tarawally
fix(sprint-1): resolve ReferenceError by declaring variable before use
6011f82
@Tarawally
fix(sprint-1): resolve TypeError by checking array bounds before acce…
dd6fc22 
…ssing properties
@Tarawally
docs(sprint-1): add concise answers for percentage change interpretation
cdd71c3
@Tarawally
docs(sprint-1): add concise answers for time format interpretation
802c33a
@Tarawally
docs(sprint-1): add concise answers for currency conversion interpret…
df747db 
…ation
@Tarawally
chore: format Sprint-1 files with Prettier
34cd9f5
@Tarawally Tarawally added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Nov 3, 2025
@Tarawally Tarawally linked an issue Nov 3, 2025 that may be closed by this pull request
03 Complete Sprint 1 Coursework Tarawally/My-Coursework-Planner#41
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

03 Complete Sprint 1 Coursework

1 participant

@Tarawally