Skip to content
This repository has been archived by the owner on Aug 1, 2022. It is now read-only.

feat: Scheduler BasicAbility & SDK #79

Merged
merged 12 commits into from
Jun 28, 2022
Merged

feat: Scheduler BasicAbility & SDK #79

merged 12 commits into from
Jun 28, 2022

Conversation

gavin1ee
Copy link
Contributor

No description provided.

@gavin1ee gavin1ee requested a review from zhenlei520 June 28, 2022 07:00
@gavin1ee gavin1ee requested a review from zhenlei520 June 28, 2022 08:33
zhenlei520
zhenlei520 approved these changes Jun 28, 2022
View reviewed changes
Copy link
Contributor

@zhenlei520 zhenlei520 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@doddgu Need to pay attention to HttpMethods

src/BasicAbility/Masa.BuildingBlocks.BasicAbility.Scheduler/Enum/HttpMethods.cs Show resolved Hide resolved
doddgu
doddgu approved these changes Jun 28, 2022
View reviewed changes
Copy link
Contributor

@doddgu doddgu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@doddgu doddgu merged commit c19f0f8 into main Jun 28, 2022
@zhenlei520 zhenlei520 deleted the feature/schedulerBasicAbility branch June 30, 2022 01:09
@zhenlei520 zhenlei520 mentioned this pull request Jul 28, 2022
Closed
9 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@doddgu doddgu doddgu approved these changes

@zhenlei520 zhenlei520 zhenlei520 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@gavin1ee @doddgu @zhenlei520