Skip to content
This repository has been archived by the owner on Mar 12, 2020. It is now read-only.

Show teachers their plan constraints in the assignment creation screen #1580

Open
BenEmdon opened this issue Sep 18, 2018 · 4 comments
Open

Comments

@BenEmdon
Copy link
Contributor

Feature request ✨

This was initially implemented in #924
And reverted in #944

@stale
Copy link

stale bot commented Dec 12, 2018

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the stale label Dec 12, 2018
@BenEmdon BenEmdon removed their assignment Jun 10, 2019
@stale stale bot removed the stale label Jun 10, 2019
@BenEmdon
Copy link
Contributor Author

@d12 we should try achieve this as part of the work being done with assignment setup. This should be a small piece of work.

@d12
Copy link
Contributor

d12 commented Jun 26, 2019

🙌 Adding to the project board!

@d12 d12 added this to Backlog in GitHub Classroom Jun 26, 2019
@spinecone
Copy link
Contributor

I've been poking around the original PR, the revert, and slack discussion around them, and it looks like what happened was:

  • Private repos #924 was deployed which started causing some kind of error
  • Fix method call #943 was created because 924 was using an incorrect method name (private_repos_available? rather than has_private_repos_available?)
  • 943 didn't deploy correctly for some reason (the heaven log has already been pruned so I can't see what the error was 😭 ) so we just reverted the whole PR with Revert "Private repos" #944

Does that sound accurate? If so I might attempt what 943 was originally trying to do and just fix the method name, and see if that works 🤞 .

@spinecone spinecone self-assigned this Aug 21, 2019
@spinecone spinecone moved this from Backlog to Done in GitHub Classroom Aug 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
GitHub Classroom
  
Shipped
Development

No branches or pull requests

3 participants