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

Fix2582 corporate workshop costs #2626

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

maneesha
Copy link
Contributor

@maneesha maneesha commented Apr 3, 2024

Fixes #2582 by removing any direct references to workshop cost, and only referencing The Carpentries website.

@maneesha
Copy link
Contributor Author

maneesha commented Apr 3, 2024

I will come back to this to fix those linting errors. @pbanaszkiewicz If I do so, would you recommend running additional migrations or deleting the migrations in this PR and replacing them?

@pbanaszkiewicz
Copy link
Contributor

@maneesha There's a good chance that if you fix this kind of error in the both models.py files, you won't need a new migration.

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.

Text about costs for corporate/for-profit workshops
2 participants