Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

remove the ignorance of testing #26

Closed
wants to merge 1 commit into from
Closed

Conversation

umarmughal824
Copy link

@umarmughal824 umarmughal824 commented Apr 20, 2021

Pre-Flight checklist

  • Screenshots and design review for any changes that affect layout or styling
    • Desktop screenshots
    • Mobile width screenshots
    • Tag @Ferdi or @pdpinch for review
  • Migrations
    • Migration is backwards-compatible with current production code
  • Testing
    • Code is tested
    • Changes have been manually tested
  • Settings
    • New settings are documented and present in app.json
    • New settings have reasonable development defaults, if applicable

What are the relevant tickets?

fixes #20

What's this PR do?

(Required)

How should this be manually tested?

(Required)

Where should the reviewer start?

(Optional)

Any background context you want to provide?

(Optional)

Screenshots (if appropriate)

(Optional)

What GIF best describes this PR or how it makes you feel?

(Optional)

* feat: switching from mongo to mysql for git logs
* feat: add SYSADMIN_MAX_GIT_LOGS_THRESHOLD threshold for CourseGitLog
@umarmughal824 umarmughal824 deleted the umar/edx-sysadmin-20 branch May 11, 2021 09:30
@umarmughal824
Copy link
Author

This got accidentally deleted with the forced commit so have opened up a new PR for it.

#30

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Refactor and pass test_sysadmin tests
1 participant