Skip to content
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

Fixed task is created again when adding the task #2356

Merged
merged 1 commit into from Jan 5, 2022
Merged

Fixed task is created again when adding the task #2356

merged 1 commit into from Jan 5, 2022

Conversation

SaidBySolo
Copy link
Member

@SaidBySolo SaidBySolo commented Jan 5, 2022

Before this PR, even if a task or a future was received, the task was created and registered once again.
When a user gives a task or future, only the task is added without creating a task or future again.

@SaidBySolo SaidBySolo requested a review from a team as a code owner January 5, 2022 07:08
@SaidBySolo SaidBySolo changed the title Fixed the task is created again when registering the task Fixed the task is created again when add the task Jan 5, 2022
@SaidBySolo SaidBySolo changed the title Fixed the task is created again when add the task Fixed task is created again when adding the task Jan 5, 2022
@ahopkins ahopkins changed the base branch from main to task-name-3.7 January 5, 2022 07:22
@ahopkins ahopkins requested a review from a team as a code owner January 5, 2022 07:22
@ahopkins
Copy link
Member

ahopkins commented Jan 5, 2022

@SaidBySolo I changed the base of this to target another PR that is going into the LTS. Can you resolve that conflict?

@SaidBySolo
Copy link
Member Author

SaidBySolo commented Jan 5, 2022

@SaidBySolo I changed the base of this to target another PR that is going into the LTS. Can you resolve that conflict?

Sure :D

@SaidBySolo
Copy link
Member Author

@ahopkins Done!

@ahopkins
Copy link
Member

ahopkins commented Jan 5, 2022

Thank you! 🎊

@ahopkins
Copy link
Member

ahopkins commented Jan 5, 2022

Okay... so I think we have some commits in here that we do NOT want to include. This is probably because you (reasonably) branched off main. Can we pull those out?

image

We do not want these commits yet.

@ahopkins
Copy link
Member

ahopkins commented Jan 5, 2022

Awesome! Thanks

@ahopkins ahopkins merged commit f0e30b1 into sanic-org:task-name-3.7 Jan 5, 2022
@SaidBySolo SaidBySolo deleted the fix/add_task branch January 5, 2022 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants