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
- Project Inactivity Clarification
I'm on appwrite free plan with my account (om.thakkar@ivcventure.com). Even though throughout the week we use the project atleast once to upload dynamic content...
- [ENHANCEMENT] Use the custom Next.js dep...
Deployment adapters on Next.js are now stable! This means that we don't have to stick to the limitations of sniffing out build outputs and manually moving or mo...
- NextJS builds sudden runtime_timeout
My builds suddenly stopped working. Activating old prebuilt snapshots work, but redeploying that same code produces builds that throw runtime_timeout FRA region