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

feat(pushmsix): Bug 1817658 - Add TB to pushmsix #1108

Merged
merged 1 commit into from
Jan 10, 2025
Merged

Conversation

hneiva
Copy link
Contributor

@hneiva hneiva commented Dec 19, 2024

No description provided.

@hneiva hneiva force-pushed the hneiva/tb-pushmsix branch from 4e6b9be to 4f1dda4 Compare December 19, 2024 22:18
@hneiva hneiva requested a review from a team January 10, 2025 19:43
@hneiva hneiva force-pushed the hneiva/tb-pushmsix branch from 4f1dda4 to 3b59c2e Compare January 10, 2025 19:43
@hneiva hneiva marked this pull request as ready for review January 10, 2025 19:43
pushmsixscript/docker.d/init_worker.sh Outdated Show resolved Hide resolved
test_var_set 'CLIENT_SECRET'
export TENANT_ID=$(echo "$TENANT_ID" | base64 -d)
export CLIENT_ID=$(echo "$CLIENT_ID" | base64 -d)
export CLIENT_SECRET=$(echo "$CLIENT_SECRET" | base64 -d)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we have values for these already, or are we waiting on someone from Thunderbird or elsewhere to provide them?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, already added them to SOPS.

@hneiva hneiva force-pushed the hneiva/tb-pushmsix branch from 3b59c2e to bec97bc Compare January 10, 2025 20:46
@hneiva hneiva requested a review from bhearsum January 10, 2025 20:54
@hneiva hneiva merged commit 983617f into master Jan 10, 2025
10 checks passed
@hneiva hneiva deleted the hneiva/tb-pushmsix branch January 10, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants