Skip to content

Serious Translation Tool- code review #8

@Frannerz

Description

@Frannerz

README

This was very detailed- good job!

User stories

Have you met the user stories?
Mostly- very interesting and fun to use!
Next step: work on responsiveness
Snip20231023_10

Learning outcomes

Have you met the LOs?
Yes!

UI bugs

Any bugs?
At one point, it stopped working. Not sure why (got this error: Error: Cannot read properties of undefined (reading 'translated')) even though I put things in. Maybe make it clearer.
Screenshot_2023-10-23_at_19 27 30

Instructions

Instructions were very straight forward.

File structure

FS, straight forward (not sure why css was in same folder as js? but clear apart from that).

Flow of control

Code was clear to follow.

Naming

Variables and functions named clearly.

Readability

Very readable code. So sorry about ours!!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions