Summary of merge requests for opencraft/documentation/public
from 2025-03-18
Merge request: docs: remove/replace mentions of OCIM
Description: ## Description
This PR remove or replaces mentions of OCIM in our public handbook
Testing instructions
Please provide detailed step-by-step instructions for testing this change.
Other information
Private Ref : BB-9386
Author: Kaustav Banerjee
For more details visit: #603.
Merge request: fix: Accidental early return on zoom account loop.
Description: This pull request fixes a bug in the zoom migration code where only the first zoom account might be honored due to an early return.
JIRA tickets: Log in - OpenCraft
Testing instructions:
make install_prereqs
- Copy all the CI variables locally in an .env file (not because .env is respected but because it isn’t tracked by source control) in this manner:
export GOOGLE_API_CREDENTIALS=$(
cat <<'END_HEREDOC'
{
"creds_or_token": {"token": "token-here", "refresh_token": "refresh-token-here", "token_uri": "https://oauth2.googleapis.com/token", "client_id": "client-id-here", "client_secret": "secret-here", "scopes": ["https://www.googleapis.com/auth/drive.file", "https://www.googleapis.com/auth/drive.metadata"], "expiry": "expiry-date"},
"scopes": ["https://www.googleapis.com/auth/drive.file", "https://www.googleapis.com/auth/drive.metadata"]
}
END_HEREDOC
)
source .env
to load the CI variables (make sure you set them all)- Add print statements to verify the loop
- Comment out the email line, since the firewall will prevent this, apparently
make migrate_zoom_meetings
- Once verified, delete the .env file
Preferably, if you have access to some, use test credentials instead of the real CI ones. Or trust me bro, it’s a one line obvious fix. How broken could it be?
Author: Fox Danger Piacenti
For more details visit: #602.
Merge request: docs: Clarify product team review role.
Description: This MR clarifies the role of product in the discovery process, and makes it clear that its involvement is not merely for adding a single line item concerning the design phase, but is also critical to the format and scope of the document.
JIRA tickets: Log in - OpenCraft
Discussions: Updates to Discovery Instructions - #11 by Fox
Merge deadline: “None” if there’s no rush, “ASAP” if it’s critical, or provide a specific date if there is one.
Testing instructions:
- Check the built version of the documentation to make sure the formatting is sensible.
Author: Fox Danger Piacenti
For more details visit: #601.
Merge request: [STAR-2983] Update info about non-billable accounts to reflect recent category changes.
Description: In the context of STAR-3920 we updated the categorization of non-billable cell and non-billable non-cell accounts; see this comment for details.
This PR updates the handbook to reflect the changes.
It also fixes some typos and removes some outdated information found along the way.
Testing
- Verify that there are no formatting issues in rendered changes.
Tickets
Reviewers
- @gabrieldamours
Author: Tim Krones
For more details visit: #600.