Skip to content

Commit 9269163

Browse files
author
justicepelteir
committed
add room for improvement section
1 parent fc44b18 commit 9269163

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

README.md

+12
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,18 @@ This application was built as a code challenge, and is my first foray into React
7272
#### Local Notifications
7373
- Local non-app foreground notifications appear on utilized iPhone for development, but not on other iPhones. I have pinpointed it to be due to Apple permissions. Expo-permissions has been recently deprecated and I am learning to implement its replacement.
7474

75+
## Room for Improvement
76+
77+
- Outsource much of the functionality within App.js as separate imported components.
78+
- Learn expo-permissions replacement and implement it since expo-permissions has been deprecated for ios notification permissions. This also should help resolve the issue of local notifications not displaying in the app foreground.
79+
- Implement ability to scroll on reminder list screen when reminders reach the point of spilling out of the screen itself.
80+
- Eliminate undefined, null, and dash mark on reminder preview in the create reminder form when no input has been entered.
81+
- Transform current 24-hour time format to AM/PM time format, or create an option to switch between them.
82+
- Make reminders actually set local notifications to the inputted scheduled reminder date/time instead of five seconds from hitting submit button.
83+
- Add ability to edit already created reminders inputed text and scheduled date/time.
84+
- Add ability to schedule reminders as repeating events.
85+
- Implement TDD, since development speed was prioritized in this project's initial iteration.
86+
7587
## How to View and Interact with this project
7688

7789
To view and interact with this project follow the steps below:

0 commit comments

Comments
 (0)