Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove `robfig/cron/v3 v3.0.1` from `go.mod`
- Remove `newSchedule()` and `resetSchedule()` functions from `services/actions/init.go`
- Change return type of `CreateScheduleTask()` from `(int, error)` to `error` in `services/actions/notifier_helper.go`
- Add `services/actions/schedule_tasks.go` file
- Add `StartScheduleTasks()` and `startTasks()` functions to `services/actions/schedule_tasks.go`
- Register `start_schedule_tasks` task in `services/cron/tasks_actions.go`

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

- Add a new function `GetSchedulesMapByIDs` to retrieve schedules by ID
- Create new schedule and schedule spec when creating a schedule task
- Delete schedule specs when deleting a schedule
- Add a new model `ActionScheduleSpec` for schedule specs

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

- Rename v245.go to v246.go
- Add a new migration for action schedule table addition

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

- Rename the `Webhook org_id` column to `owner_id`
- Add a new migration for creating an `action schedule` table

…nability

- Rename `jobs` to `workflows` in `notifier_helper.go`
- Fix the `if` statement to properly continue in `notifier_helper.go`
- Remove unnecessary `else` block in `notifier_helper.go`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

…on v246

- Remove `EntryIDs` field from `ActionSchedule` struct in `schedule.go`
- Remove `EntryIDs` field from migration v246 in `v1_20` folder

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Rename `jobs` to `workflows` for clarity
- Simplify error handling in line 14
- Simplify error handling in line 19
- Remove unnecessary else statement in line 22

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove the license for `github.com/robfig/cron/v3`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Update `CreateScheduleTask` to use the passed context instead of `db.DefaultContext`
- Update `DeleteScheduleTaskByRepo` to use the passed context instead of `db.DefaultContext`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Delete the `ifNeedApproval` function from `notifier_helper.go`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Add a license to the `go-licenses.json` file

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove a line that had no newline at the end of the file

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Add an import statement in `schedule_spec_list.go`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Change `run` initialization to a pointer
- Add function `ifNeedApproval` for deciding if a user needs approval
- Modify `handleWorkflows` to use `ifNeedApproval` function
- Add `NeedApproval` field to `run` in `handleWorkflows` function

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Fix a typo in the `CreateScheduleTask` function
- Remove unnecessary code from the `CreateScheduleTask` function

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Add `xorm: "index"` tag to `RepoID` and `ScheduleID` fields in `ActionScheduleSpec`
- Remove `RepoID` and `ScheduleID` fields from the model struct

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove the function `CreateScheduleTask` from the `notifier_helper.go` file
- Add the function `CreateScheduleTask` to the `schedule_tasks.go` file, which creates a scheduled task from a cron action schedule and a spec string. It creates an action run based on the schedule, inserts it into the database, and creates commit statuses for each job.

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Change in variable assignment for `specs` in `startTasks` function

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove the handleWorkflows function and replace it with a new version that has a different signature and takes more arguments.

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Co-authored-by: Jason Song <i@wolfogre.com>

- Update the `now` variable to truncate to the nearest minute
- Change the calculation of `next` to use `now.Add(-1)` instead of `now`
- Replace `next.Sub(now) <= 60` with `schedule.Next(now.Add(-1)).Equal(now)`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Change the initialization of `crons` to allocate capacity upfront, improving performance.
- Remove an unused variable definition.
- Add a new function `startTasks` with `opts` parameter.

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove debug log for workflow content
- Simplify check for empty crons slice
- Remove unnecessary if statement
- Simplify return statement in handleSchedules function

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove `GetAll` field from `FindSpecOptions` struct
- Add `FindAllSpecs` function to retrieve all matching schedule specs
- Modify `StartScheduleTasks` to use `FindAllSpecs` instead of `FindSpecs`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Rename file v246.go to v248.go in models/migrations/v1_20/ directory.

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

…ully

- Add a check for `len(rows)` before executing `CreateScheduleTask` function
- Remove a redundant check for `len(rows)` in `CreateScheduleTask` function

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove the function `FinAllSpecs` from `schedule_spec_list.go`
- Change the page size to 50 in `schedule_spec_list.go`
- Retrieve specs in pages in `schedule_tasks.go`
- Create a schedule task for each spec whose schedule is due
- Check for invalid schedules before creating a schedule task

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove `registerScheduleTasks` function
- Add a new implementation of `registerScheduleTasks` function

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- File `v248.go` was renamed to `v249.go`.

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

- Rename `v249.go` to `v250.go` in `models/migrations/v1_20`

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

…and readability

No summary comments, as there are no changes in the git diff.

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

- Remove code that fetched all schedule events and only keep the schedule event detection
- Simplify the schedule event detection code

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi.Wu <appleboy.tw@gmail.com>

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>

Signed-off-by: appleboy <appleboy.tw@gmail.com>

Signed-off-by: appleboy <appleboy.tw@gmail.com>

- Remove loop that creates schedule tasks for each cron and spec

Signed-off-by: appleboy <appleboy.tw@gmail.com>

Signed-off-by: appleboy <appleboy.tw@gmail.com>

Signed-off-by: appleboy <appleboy.tw@gmail.com>

- Import "time" and "github.com/gogs/cron" packages
- Add GetReposMapByIDs function to retrieve repos by given ID slice
- Add comments and refactor CreateScheduleTask function
- Import "context" and "github.com/gogs/cron" in schedule_spec.go
- Add new fields and Parse function to ActionScheduleSpec struct
- Add UpdateScheduleSpec function
- Update LoadSchedules function in schedule_spec_list.go
- Add Next field in FindSpecOptions struct
- Update models/migrations/v1_20/v256.go
- Refactor startTasks function in schedule_tasks.go

Signed-off-by: appleboy <appleboy.tw@gmail.com>

- Add ActionScheduleSpec and ActionSchedule with RepoID in DeleteRepository function

Signed-off-by: appleboy <appleboy.tw@gmail.com>

- Add a missing error return statement in `handleSchedules` function

Signed-off-by: appleboy <appleboy.tw@gmail.com>

- Remove `index` tag from `WorkflowID` in `ActionSchedule` struct
- Add `index` tag to `RepoID` and `ScheduleID` in `AddActionScheduleTable` function

Signed-off-by: appleboy <appleboy.tw@gmail.com>

- Rename migration file from `v256.go` to `v257.go`

Signed-off-by: appleboy <appleboy.tw@gmail.com>

Signed-off-by: appleboy <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>

- Add `github.com/gogs/cron` to the `go.mod` file
- Remove `GetAll` from `FindScheduleOptions` struct in `schedule_list.go`
- Replace `if !opts.GetAll` with `if !opts.ListAll` in `FindSchedules` function in `schedule_list.go`
- Remove `handleSchedules` function from `notifier_helper.go`
- Modify `notify` function in `notifier_helper.go` to include `schedules` in `DetectWorkflows` function calls
- Replace `workflows` with `detectedWorkflows` in `handleWorkflows` function in `notifier_helper.go`
- Add `handleSchedules` function in `notifier_helper.go` to handle detected workflows and create schedule tasks
- Modify `handleWorkflows` function in `notifier_helper.go` to cancel running jobs if the event is push
- Modify `notify` function in `notifier_helper.go` to call `handleWorkflows` with `detectedWorkflows` instead of `workflows`

Signed-off-by: appleboy <appleboy.tw@gmail.com>

- Add error return statements in the `startTasks` function in two places.

Signed-off-by: appleboy <appleboy.tw@gmail.com>

Co-authored-by: wxiaoguang <wxiaoguang@gmail.com>

…boy-schedule