This is in regards to https://github.com/appwrite/console/pull/553 @Steven Basically there are 2 problems:
- Unit tests are validating Incorrectly (Investiging it)
- A regex pattern to match the cron expression is incresingly complex, Instead i have broken it down and tested the regex of indiviidual parts of the cron, while combing it however there are edges cases that are popping up
Paths I can Take:
- I can continue on trying to debug this regex expression (need some help)
- A different route would be, us relying on this function instead of a single regex expresssion for client side validation, as far as my tests went this function is working correctly (not sure if this is a good idea, but it would work well)
can we simplify the cron syntax? covering the usual expressions should be good enough
as for testing...i would want to make sure we're testing with the input because it seems like it may not accept all regex patterns
i was trying to cover all the test cases to be accepted, they are extensive. i can simplify it to cover a subset of the tests
like i said, we could switch to manual validation if that is okay
Recommended threads
- Generate CSR
How do I generate a CSR for my domain host? They are asking me to generate one for my hoosting here on appwrite
- Appwrite Push Notifications: "Unknown er...
Hi all, I'm running into an issue with Appwrite push notifications in my web project. When I try to manually send a notification to a user from the Appwrite we...
- Realtime fails to connect/reconnect
I have been having trouble with my appwrite realtime connection for a few days now. Before the code that was working fine now is giving failed to connect errors