@team
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
Tickets
Reviewers
Author: Tim Krones
For more details visit: #600.