Application main page, remove top bar #29

2025-02-23T15:40:50-08:00

djwesty commented

2025-02-23 15:40:50 -08:00

(Migrated from github.com)

Addresses Issue #24

This work addresses the issue we had of the expo router title bar simply saying "index". The issue was originally to remove the bar, but instead I decided to keep it and just change the name to our app name. I instead removed the redundant title within the page itself

I also added coverage to gitignore, which is automatically generated by jest when you test for code coverage.

Also some minor ts/js tweaks to set some code examples

Addresses Issue #24 This work addresses the issue we had of the expo router title bar simply saying "index". The issue was originally to remove the bar, but instead I decided to keep it and just change the name to our app name. I instead removed the redundant title within the page itself I also added `coverage` to gitignore, which is automatically generated by `jest` when you test for code coverage. Also some minor ts/js tweaks to set some code examples

MantashaNoyela (Migrated from github.com) reviewed 2025-02-23 15:40:50 -08:00

Vutukuri15 (Migrated from github.com) approved these changes 2025-02-23 15:55:14 -08:00

Vutukuri15 (Migrated from github.com) left a comment

Approved! Nice work!

.gitignore

Vutukuri15 (Migrated from github.com) commented

2025-02-23 15:51:27 -08:00

Thanks for adding the test coverage config! It will be really helpful.

app/_layout.tsx

Vutukuri15 (Migrated from github.com) commented

2025-02-23 15:52:06 -08:00

I see you changed the index to the page title. That makes the page more readable.