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

Add ability to test A+ integration locally #70

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

mikaelGusse
Copy link
Contributor

@mikaelGusse mikaelGusse commented Jan 20, 2025

Description

What?

  • Adds checks and error handling which allows for A+ and Radar integration to be tested locally

Why?

  • Helps with adding features regarding A+ integration. For example identifying teachers/TAs

How?

  • Added more lenient error handling for fetching template code, additionally multiple checks to skip asynchronous tasks when running locally, since setting up the Celery workers is a pain (especially on mac)

Fixes #69

Testing

Ran E2E tests and tested features manually

Fetching template code might not work currently locally

Think of what is affected by these changes and could become broken

Documentation

Added documentation on how to develop with A+ locally

Clean up your git commit history before submitting the pull request!

@mikaelGusse mikaelGusse force-pushed the master branch 2 times, most recently from 2c4ee00 to 2449c2c Compare January 21, 2025 13:11
- Adds checks and error handling which allows for A+ and Radar integration to be tested locally
@mikaelGusse mikaelGusse marked this pull request as ready for review January 21, 2025 13:36
@Harman-Aalto
Copy link
Contributor

Excellent work. This should help with local development.

@Harman-Aalto Harman-Aalto merged commit 08314f1 into apluslms:master Jan 22, 2025
2 checks passed
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.

Create mock A+ api for local testing
2 participants