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
- Fulltext index creation fails due to Inn...
I'm running a self-hosted Appwrite 1.8.1 instance and encountering an issue when creating fulltext indexes on a collection. **The Problem:** Fulltext index cre...
- Unable To Use Root Domain In Appwrite
Since Appwrite Manages my DNS Completely, I tried Pointing the Root Domain (**getmyself.app**) to Github Pages, It failed So I tried Using Appwrite Sites, also ...
- Report: fetch failed
https://github.com/appwrite/appwrite/issues/10989 please assign this issue to me