8000 🐛 Bug Report: Schedule function with minutes does not work · Issue #5719 · appwrite/appwrite · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
8000

🐛 Bug Report: Schedule function with minutes does not work #5719

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
2 tasks done
jbdujardin opened this issue Jun 16, 2023 · 16 comments · Fixed by appwrite/console#553
Closed
2 tasks done

🐛 Bug Report: Schedule function with minutes does not work #5719

jbdujardin opened this issue Jun 16, 2023 · 16 comments · Fixed by appwrite/console#553
Assignees
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed product / console Console, UI and UX issues product / functions Fixes and upgrades for the Appwrite Functions.

Comments

@jbdujardin
Copy link
jbdujardin commented Jun 16, 2023

👟 Reproduction steps

I create and deploy a function with cron schedule 30 1 * * * :

{
            "$id": "update-fixtures",
            "name": "Update fixtures",
            "runtime": "dart-2.17",
            "path": "functions/update-fixtures",
            "entrypoint": "lib/main.dart",
            "ignore": [
                ".packages",
                ".dart_tool"
            ],
            "execute": [],
            "events": [],
            "schedule": "30 1 * * *",
            "timeout": 60
},

The function is deployed and build successfully but the schedule is not active despite the cron is visible on the function settings :

Capture d’écran 2023-06-16 à 23 55 35

If I try to modify the value on the console, there is an error :

Capture d’écran 2023-06-16 à 23 57 39

I also try to edit the value on the json and redeploy the function but no schedule active ☹️

If I set * * * * * or 0 1 * * * , there is no problem, only when minutes are other than * or 0.

👍 Expected behavior

The function is executed with the cron value on the Schedule part (with minutes on the cron value).

👎 Actual Behavior

The function is not executed and the schedule is not active (no clock on the bottom right).

🎲 Appwrite version

Version 1.3.x

💻 Operating system

MacOS

🧱 Your Environment

Appwrite version 1.3.7
CLI version 2.0.2

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@jbdujardin jbdujardin added the bug Something isn't working label Jun 16, 2023
@stnguyen90 stnguyen90 self-assigned this Jun 18, 2023
@stnguyen90 stnguyen90 added product / functions Fixes and upgrades for the Appwrite Functions. product / console Console, UI and UX issues labels Jun 18, 2023
@sunjianan9900
Copy link

i got same problem
The strange thing is that after I set up the json file, open the console to view, the clock icon did appear, but when I clicked into the FC to view the timing (no changes), and then return to the FC home page, the clock icon disappeared

@sunjianan9900
Copy link

This is Schrödinger's cat, you set up the json file, deploy function, and then do not enter the setting page of this function, other pages can be viewed (such as logs), the timing will not be canceled, but as long as you enter the setting page, the timing will expire

@sunjianan9900
Copy link

This should be a display bug, I just tested the setting to execute every 10 minutes (via json), although the page will have a clock for a while, and then not, but the methods are executed on time (check the log). You can set a more frequent timing to see what happens. (After the first automatic execution, the clock marker didn't disappear anymore)

@stnguyen90 stnguyen90 added help wanted Extra attention is needed good first issue Good for newcomers labels Jun 30, 2023
@stnguyen90
Copy link
Contributor

We're going to focus this issue on the incorrect validation error.

@stnguyen90 stnguyen90 removed their assignment Jun 30, 2023
@Sachin24704
Copy link

Hi , so the incorrect validation is due to regex pattern.

@Sachin24704
Copy link

PR was created for this issue. Please review

@stnguyen90
Copy link
Contributor

@Sachin24704 would you please link to this issue from the PR so it shows up here?

@stnguyen90 stnguyen90 added this to the 1.3.8 milestone Jul 6, 2023
@Sachin24704
Copy link
Sachin24704 commented Jul 7, 2023

the PR is in the console repo - appwrite/console#473 -hope this helps. I was not able to link it here in this repo. Thanks

@antonioorct
Copy link

Is there any work-around for this? The issue's open for a long time and I'm having huge issues on production due to this.

@stnguyen90
Copy link
Contributor

Closing and unassigning due to inactivity.

@stnguyen90 stnguyen90 modified the milestones: 1.4.0, 1.4.2 Aug 31, 2023
@stnguyen90 stnguyen90 removed this from the 1.4.2 milestone Aug 31, 2023
@stnguyen90
Copy link
Contributor

Is there any work-around for this? The issue's open for a long time and I'm having huge issues on production due to this.

@sunjianan9900 the workaround is to use your browser dev tools to remove the client-side pattern validation.

@KanniShashankh
Copy link

@stnguyen90 Can i be assigned this issue?

@stnguyen90
Copy link
Contributor

@KanniShashankh, assigned! Thanks for your interest! 🙏

@KanniShashankh
Copy link

@KanniShashankh, assigned! Thanks for your interest! 🙏

Are we going to support all types of cron syntax? Because the regex pattern would be pretty verbose for covering all cases

@stnguyen90
Copy link
Contributor

@KanniShashankh let's just focus on fixing the bugs and supporting standard syntax

@KanniShashankh
Copy link

@stnguyen90 Please review i have mentioned the PR here, Thank you

@stnguyen90 stnguyen90 linked a pull request Sep 15, 2023 that will close this issue
@stnguyen90 stnguyen90 moved this from In Progress to Code Review in 1.4 release Oct 13, 2023
@github-project-automation github-project-automation bot moved this from Code Review to Done in 1.4 release Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed product / console Console, UI and UX issues product / functions Fixes and upgrades for the Appwrite Functions.
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

6 participants
0