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
- How to properly implement custom domain?...
- Site deployment does not work.
I can change a file, deploy, change does not take effect. Delete file in repo, push changes,deploy. Old file remains undeleted Duplicate file, rename it, push,...
- Appwrite DNS nameservers return SERVFAIL...
My custom domain demomanasa.online (and all its subdomains) stopped resolving with ERR_NAME_NOT_RESOLVED for all visitors. I have traced the failure to Appwrite...