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

chore: barcalendar Redwood updates #529

Merged
merged 1 commit into from
Jun 25, 2024

Conversation

cmltaWt0
Copy link
Contributor

Changes:

  • removed Redwood from future releases
  • changed CURRENT and EDX version of everything

@feanil I didn't change Mongo, MySQL, Elasticsearch, Redis, Ruby for EDX. I need an info about the current software versions for edx.org.

@openedx-webhooks openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Jun 25, 2024
@openedx-webhooks
Copy link

Thanks for the pull request, @cmltaWt0! 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.

@feanil
Copy link
Contributor

feanil commented Jun 25, 2024

@robrap FYI, if you want to update the edx versions on the support windows spreadsheet, this is the place to do it. @cmltaWt0 I think we can just merge this without those updates and I'll re-run the code on the spreadsheet to update.

@feanil feanil merged commit f5466b7 into openedx:master Jun 25, 2024
4 checks passed
@openedx-webhooks
Copy link

@cmltaWt0 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

@robrap
Copy link
Contributor

robrap commented Jun 25, 2024

Thanks @feanil. Was there a specific context for this inform, or just a very general inform?

@feanil
Copy link
Contributor

feanil commented Jun 25, 2024

Very general, this info is out of date and if you're relying on it, you may want to update, if you're no longer using this let me know and we can drop the data from the sheet.

@robrap
Copy link
Contributor

robrap commented Jun 25, 2024

@feanil: When you say:

... let me know and we can drop the data from the sheet
What are you referring to? Are you referring to specific data, or are you talking about the entire Support Window spreadsheet? The spreadsheet is certainly useful.

Related, that spreadsheet has an IDA and Upgrade status tab that seems quite out-of-date. How is that updated, and who uses this?

FYI: @jristau1984

@feanil
Copy link
Contributor

feanil commented Jun 25, 2024

So there is a 2U specific copy of this but also the public one under Axim control. There is also this script that updates the data with the latest info for 2U and for the community supported release. I'm wondering if it is useful to still have the code in place that annotates the 2U specific states on the sheets. If it is still useful, please update the state in the code so that it matches the current state of your systems. If it is not useful and is going to become out of date, let me know and I'll just remove the 2U specific bits rather than have out of date information.

@robrap
Copy link
Contributor

robrap commented Jun 25, 2024

@feanil: Did you mean for "this script" to be a link?

@feanil
Copy link
Contributor

feanil commented Jun 25, 2024

No, it's the script that this PR is making updates to.

@robrap
Copy link
Contributor

robrap commented Jun 25, 2024

@feanil: Sorry if I am just dense.

I'm wondering if it is useful to still have the code in place that annotates the 2U specific states on the sheets.

Can you link to code or provide more specifics of what this refers to, and what is 2U specific. I don't see any code comments about "2U". Thank you.

@feanil
Copy link
Contributor

feanil commented Jun 26, 2024

@robrap
Copy link
Contributor

robrap commented Jun 26, 2024

Thanks @feanil. I can determine whether this information is useful internally, but is it useful to the community at all as well while doing upgrades? Or would this information be updated too late for it to ever be useful, because those details would have come in through another mechanism?

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

Successfully merging this pull request may close these issues.

4 participants