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

Control registration #156

Merged
merged 8 commits into from
Dec 31, 2021
Merged

Conversation

Lourenzutti
Copy link
Contributor

@trevorcampbell

I added a registration deadline check so the deadlines are not overridden after this deadline.
Could you take a look?

I also removed a couple of unused libraries.

Thanks.

rudaux/flows.py Outdated Show resolved Hide resolved
rudaux/flows.py Outdated Show resolved Hide resolved
rudaux/submission.py Outdated Show resolved Hide resolved
rudaux/flows.py Outdated Show resolved Hide resolved
@trevorcampbell
Copy link
Contributor

trevorcampbell commented Dec 31, 2021

Two changes requested above, and one question (probably will cause more changes).

Then it should be ready to merge!

Make sure you link the relevant issue so that when the PR merges it gets closed too.

@trevorcampbell
Copy link
Contributor

Looks good to me now! Are you able to merge or do I have to do that?

@Lourenzutti Lourenzutti merged commit 2e55537 into prefect-by-assignment Dec 31, 2021
@Lourenzutti
Copy link
Contributor Author

Done!

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.

Rudaux autoext does weird things when reg dates are updated erroneously
2 participants