Skip to content
This repository was archived by the owner on May 6, 2024. It is now read-only.

Conversation

qasimgulzar
Copy link

@qasimgulzar qasimgulzar commented Jan 29, 2023

  1. Removed broken repositories
  2. Updated boto version

@openedx-webhooks openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Jan 29, 2023
@openedx-webhooks
Copy link

openedx-webhooks commented Jan 29, 2023

Thanks for the pull request, @qasimgulzar! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

  • supporting documentation
  • Open edX discussion forum threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

@mphilbrick211
Copy link

Hi @qasimgulzar! Just checking in to see if you plan on pursuing this pull request? If so, you'll need to submit a CLA form as mentioned above.

@mphilbrick211 mphilbrick211 added the inactive PR author has been unresponsive for several months label Apr 4, 2023
@mphilbrick211 mphilbrick211 added closed inactivity PR was closed because the author abandoned it and removed inactive PR author has been unresponsive for several months labels Apr 11, 2023
@mphilbrick211
Copy link

Hi @qasimgulzar - I am closing this pull request for now. If you'd like to pursue in the future, we can reopen. Thanks!

@qasimgulzar
Copy link
Author

@mphilbrick211 I have submitted CLA could you please reopen this PR?

@mphilbrick211 mphilbrick211 reopened this Mar 21, 2024
@mphilbrick211
Copy link

Hi @qasimgulzar! Thanks for submitting your CLA form. If you re-run the checks the CLA check should turn green.

Also, when you get a moment, could you please add a description of your change(s) to the top of the pull request? Thanks!

@mphilbrick211 mphilbrick211 removed the closed inactivity PR was closed because the author abandoned it label Mar 21, 2024
@qasimgulzar qasimgulzar marked this pull request as ready for review March 26, 2024 10:47
@qasimgulzar
Copy link
Author

@mphilbrick211 could you please help reviewing this PR?

@mphilbrick211
Copy link

@mphilbrick211 could you please help reviewing this PR?

Hi @qasimgulzar! Yes, I will look into this for you.

@pomegranited
Copy link
Contributor

Hi @qasimgulzar @mphilbrick211 I didn't think we were maintaining Ironwood anymore, since we're at Quince now?

@pomegranited pomegranited self-assigned this Mar 28, 2024
@pomegranited
Copy link
Contributor

Closing this PR as Ironwood is unmaintained.

@openedx-webhooks
Copy link

@qasimgulzar Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

1 similar comment
@openedx-webhooks
Copy link

@qasimgulzar Even though your pull request wasn’t merged, please take a moment to answer a two question survey so we can improve your experience in the future.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
open-source-contribution PR author is not from Axim or 2U
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants