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

Merge 2.34 code freeze #1692

Merged
merged 21 commits into from
Sep 13, 2024
Merged
Show file tree
Hide file tree
Changes from 17 commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
5138a01
[Bug] If a deeplink is malformed or missing data we show auth screen.
notandyvee Aug 1, 2024
7e68496
Show loading dialog when loading magic link.
notandyvee Aug 1, 2024
19f21bc
Bump version name and build code
wpmobilebot Sep 10, 2024
1b99c64
Update draft release notes for 2.34.
wpmobilebot Sep 10, 2024
cc63d77
Release Notes: add new section for next version (2.35)
wpmobilebot Sep 10, 2024
0ad4fbb
Update strings file for translation automation
wpmobilebot Sep 10, 2024
9b72379
Update release notes source with placeholder copy for 2.34
mokagio Sep 10, 2024
b88bf41
Merge remote-tracking branch 'origin/trunk' into andy/firebird-issue-81
mokagio Sep 10, 2024
bb90134
Update release notes source with placeholder copy for 2.34 (#1691)
mokagio Sep 10, 2024
bfea8bc
Use `RELEASE_VERSION` instead of `BUILDKITE_RELEASE_VERSION`
mokagio Sep 10, 2024
6fce8c8
Update `.configure` using latest toolkit version
mokagio Sep 10, 2024
080269b
Update `PlayStoreStrings.pot` for 2.34
wpmobilebot Sep 10, 2024
30013e5
Fix `BUILDKITE_ORGANIZATION` defined as `BUILDKITE_ORG`
mokagio Sep 10, 2024
c1c0b2d
Add missing `version` and `build_code` to `build_for_distribution`
mokagio Sep 10, 2024
18aba59
Account or different globbing behavior in Bash vs Zsh
mokagio Sep 11, 2024
30ef95b
Specify `-maxdepth` before `-name`
mokagio Sep 11, 2024
057c1b7
Make all release Buildkite command scripts executable
mokagio Sep 11, 2024
4ed1be7
Fix incorrect APK path in upload script
mokagio Sep 11, 2024
98e9ccc
Use Fastlane to decrypt secrets in prototype build step
mokagio Sep 11, 2024
3b01a75
Use full release name when creating GitHub release
mokagio Sep 11, 2024
bd79e42
[Bug] If a deeplink is malformed or missing data we show auth screen.…
notandyvee Sep 11, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 2 additions & 5 deletions .buildkite/commands/build-prototype.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,8 @@ set -eu
echo "--- :rubygems: Setting up Gems"
install_gems

echo "--- Installing Secrets"
# This is a hack until our AMI has a newer version of glibc and openSSL
./gradlew
chmod +x vendor/configure/configure
docker run -it --rm --workdir /app --env CONFIGURE_ENCRYPTION_KEY -v $(pwd):/app public.ecr.aws/automattic/android-build-image:4281c9e97b2d821df3de34c046b7c067499b35bb /bin/bash -c 'vendor/configure/configure apply'
echo "--- :closed_lock_with_key: Installing Secrets"
bundle exec fastlane run configure_apply

echo "--- :hammer_and_wrench: Build and Test"
bundle exec fastlane build_and_upload_prototype_build
8 changes: 3 additions & 5 deletions .buildkite/commands/checkout-release-branch.sh
Original file line number Diff line number Diff line change
@@ -1,14 +1,12 @@
#!/bin/bash -eu

# We expect BUILDKITE_RELEASE_VERSION to be as an environment variable, e.g. by the automation that triggers the build on Buildkite.
# It must use the `BUILDKITE_` prefix to be passed to the agent due to how `hostmgr` works, in case this runs on a Mac agents.
if [[ -z "${BUILDKITE_RELEASE_VERSION}" ]]; then
echo "BUILDKITE_RELEASE_VERSION is not set."
if [[ -z "${RELEASE_VERSION}" ]]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

I double-checked and this checkout-release-branch.sh script seems to only be called by complete-code-freeze.yml, from a step that runs on tumblr-metal.

So indeed there's no risk of the env var being filtered out by hostmgr when the script to run is passed from Mac host to VM here, because this will run on tumblr-metal and not Mac agents, so hostmgr is not even involved in the first place. 👍

echo "RELEASE_VERSION is not set."
exit 1
fi

# Buildkite, by default, checks out a specific commit.
# For many release actions, we need to be on a release branch instead.
BRANCH_NAME="release/${BUILDKITE_RELEASE_VERSION}"
BRANCH_NAME="release/${RELEASE_VERSION}"
git fetch origin "$BRANCH_NAME"
git checkout "$BRANCH_NAME"
4 changes: 2 additions & 2 deletions .buildkite/commands/release-build.sh
100644 → 100755
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@ bundle exec fastlane build_for_distribution
echo "--- :computer: Use deterministic APK name for next steps"

OUTPUT_DIR="build"
if [ "$(find "$OUTPUT_DIR/*.apk" | wc -l)" -gt 1 ]; then
if [ "$(find "$OUTPUT_DIR" -maxdepth 1 -name "*.apk" | wc -l)" -gt 1 ]; then
echo "Found more than one APK in $OUTPUT_DIR."
exit 1
fi

ORIGINAL_APK_PATH=$(ls -1 "$OUTPUT_DIR/*.apk")
ORIGINAL_APK_PATH=$(find "$OUTPUT_DIR" -name "*.apk" -maxdepth 1 | head -1)

set -x
mv "$ORIGINAL_APK_PATH" $OUTPUT_DIR/simplenote.apk
Expand Down
Empty file modified .buildkite/commands/release-create-on-github.sh
100644 → 100755
Empty file.
2 changes: 1 addition & 1 deletion .buildkite/commands/release-upload.sh
100644 → 100755
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@ bundle exec fastlane run configure_apply

echo "--- :android: Upload to Play Store"
bundle exec fastlane upload_build_to_play_store \
apk_path:"$ARTIFACT_PATH/simplenote.apk" \
apk_path:"$ARTIFACT_PATH" \
"beta:${1:-true}" # use first call param, default to true for safety
22 changes: 15 additions & 7 deletions .configure
Original file line number Diff line number Diff line change
@@ -1,27 +1,35 @@
{
"project_name": "Simplenote-Android",
"branch": "trunk",
"pinned_hash": "9b6326550a87fe0b6423a9e12f79ca92be8c9cff",
"pinned_hash": "2c5509d08ff9b9620381f4dc602fc2e5edcfcb40",
"files_to_copy": [
{
"file": "android/simplenote/Simplenote/gradle.properties",
"destination": "Simplenote/gradle.properties"
"destination": "Simplenote/gradle.properties",
"encrypt": true
},
{
"file": "android/simplenote/sentry.properties",
"destination": "sentry.properties"
"destination": "sentry.properties",
"encrypt": true
},
{
"file": "android/automattic.jks",
"destination": ".configure-files/release.jks"
"destination": ".configure-files/release.jks",
"encrypt": true
},
{
"file": "android/debug.keystore",
"destination": ".configure-files/debug_a8c.keystore"
"destination": ".configure-files/debug_a8c.keystore",
"encrypt": true
},
{
"file": "android/Simplenote/google-upload-credentials.json",
"destination": ".configure-files/google-upload-credentials.json"
"destination": ".configure-files/google-upload-credentials.json",
"encrypt": true
}
],
"file_dependencies": [

]
}
}
Binary file added .configure-files/automattic.jks.enc
Binary file not shown.
Binary file modified .configure-files/debug.keystore.enc
Binary file not shown.
Binary file modified .configure-files/google-upload-credentials.json.enc
Binary file not shown.
Binary file modified .configure-files/gradle.properties.enc
Binary file not shown.
Binary file modified .configure-files/sentry.properties.enc
Binary file not shown.
4 changes: 4 additions & 0 deletions RELEASE-NOTES.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
2.35
-----


2.34
-----

Expand Down
13 changes: 6 additions & 7 deletions Simplenote/metadata/PlayStoreStrings.pot
Original file line number Diff line number Diff line change
Expand Up @@ -11,20 +11,19 @@ msgstr ""
"Project-Id-Version: Release Notes & Play Store Descriptions\n"

#. translators: Release notes for this version to be displayed in the Play Store. Limit to 500 characters including spaces and commas!
msgctxt "release_note_0234"
msgid ""
"2.34:\n"
"Under the hood improvements\n"
msgstr ""

msgctxt "release_note_0233"
msgid ""
"2.33:\n"
"* Added support for logging in with magic links for a passwordless experience.\n"
"\n"
msgstr ""

msgctxt "release_note_0232"
msgid ""
"2.32:\n"
"• Added adaptive app icon\n"
"• Removed Sustainer plan information and upgrade UI\n"
msgstr ""

#. translators: Short description of the app to be displayed in the Play Store. Limit to 80 characters including spaces and commas!
msgctxt "play_store_promo"
msgid "Take notes, create to-do lists, capture ideas, and more."
Expand Down
2 changes: 1 addition & 1 deletion fastlane/Fastfile
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ ORGANIZATION = 'automattic'
REPO_NAME = 'simplenote-android'
GITHUB_REPO = "#{ORGANIZATION}/#{REPO_NAME}".freeze
DEFAULT_BRANCH = 'trunk'
BUILDKITE_ORG = ORGANIZATION
BUILDKITE_ORGANIZATION = ORGANIZATION
BUILDKITE_PIPELINE = REPO_NAME
APP_PACKAGE_NAME = 'com.automattic.simplenote'

Expand Down
3 changes: 3 additions & 0 deletions fastlane/lanes/build.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@

platform :android do
lane :build_for_distribution do
version = VERSION_FILE.read_version_name
build_code = build_code_current

user_friendly_output_dir = File.join(PROJECT_ROOT_FOLDER, 'build')
user_friendly_output_apk_name = "simplenote-#{version}.apk"
user_friendly_output_path = File.join(user_friendly_output_dir, user_friendly_output_apk_name)
Expand Down
2 changes: 1 addition & 1 deletion fastlane/lanes/release.rb
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@
trigger_buildkite_release_build(branch: branch_to_build, beta: false)
end

lane :create_release_on_github do |version: release_version_current, beta: true, apk_path: GRADLE_APK_OUTPUT_PATH.to_s|
lane :create_release_on_github do |version: VERSION_FILE.read_version_name, beta: true, apk_path: GRADLE_APK_OUTPUT_PATH.to_s|
create_github_release(
repository: GITHUB_REPO,
version: version,
Expand Down
1 change: 1 addition & 0 deletions fastlane/metadata/android/en-US/changelogs/default.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Under the hood improvements
1 change: 1 addition & 0 deletions fastlane/resources/values/strings.xml
Original file line number Diff line number Diff line change
Expand Up @@ -450,4 +450,5 @@
<string name="magic_link_login_with_wordpress_or_break">Or</string>
<string name="magic_link_confirm_code_enter_pass_label">Enter password</string>
<string name="magic_link_confirm_code_message">We\'ve sent a code to \n%1$s. The code will be valid for a few minutes.</string>
<string name="login_with_password_message">Enter the password for the account %1$s</string>
</resources>
4 changes: 2 additions & 2 deletions version.properties
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
versionName=2.33
versionCode=169
versionName=2.34-rc-1
versionCode=170
Loading