Skip to content
This repository has been archived by the owner on Dec 14, 2021. It is now read-only.

Remove product name from strings #992

Merged
merged 7 commits into from
May 29, 2019
Merged

Remove product name from strings #992

merged 7 commits into from
May 29, 2019

Conversation

joeyg
Copy link
Contributor

@joeyg joeyg commented May 17, 2019

Fixes #991

@joeyg joeyg requested a review from a team as a code owner May 17, 2019 18:16
Copy link
Contributor

@Pike Pike left a comment

Choose a reason for hiding this comment

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

FYI, we've stripped and fixed the Lockbox on our side in mozilla-l10n/lockwiseios-l10n@b8c623c for now.

Copy link
Contributor

@sashei sashei left a comment

Choose a reason for hiding this comment

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

I think we need to fix the FaceID usage description as well.

@devinreams
Copy link
Contributor

The resulting output of the export scripts is shown here:
mozilla-l10n/lockwiseios-l10n#4

Any other changes need up here or downstream?

@devinreams devinreams requested a review from Pike May 21, 2019 16:09
Copy link
Contributor

@sashei sashei left a comment

Choose a reason for hiding this comment

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

lgtm!

@Pike
Copy link
Contributor

Pike commented May 22, 2019

I've left some a comment on the import PR at mozilla-l10n/lockwiseios-l10n#4 (review).

In particular, there's still some "Lockbox" literal being imported back, which we'd like to get fixed still.

Copy link
Contributor

@Pike Pike left a comment

Choose a reason for hiding this comment

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

Denoting the change request

@devinreams
Copy link
Contributor

Note to future self: where is the CFBundleName coming from? I can't tell how those strings get interpolated..

@devinreams devinreams self-assigned this May 22, 2019
@Pike
Copy link
Contributor

Pike commented May 23, 2019

Note to future self: where is the CFBundleName coming from? I can't tell how those strings get interpolated..

<key>CFBundleName</key>
<string>$(PRODUCT_NAME)</string>
I think

@devinreams
Copy link
Contributor

Results of latest export based on this branch is up here (no more Lockbox!):
mozilla-l10n/lockwiseios-l10n#5

@devinreams
Copy link
Contributor

Oh I got aggressive and need to not rename the app quite so much.

Copy link
Contributor

@Pike Pike left a comment

Choose a reason for hiding this comment

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

I looked at the exported l10n strings, and they look OK, but the path change around Strings/en.lproj surprise me.

I can't pin them down to anything in this patch, though, did something else change?

@devinreams
Copy link
Contributor

I can't pin them down to anything in this patch, though, did something else change?

Thanks @Pike.

I think it would have to be when we removed and then re-added (to clean out duplicate file inclusions) in Xcode here: #1011

Copy link
Contributor

@Pike Pike left a comment

Choose a reason for hiding this comment

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

sgtm

@devinreams devinreams merged commit d6c1d2f into master May 29, 2019
@devinreams devinreams deleted the remove-product-name branch May 29, 2019 22:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix remaining Lockbox strings
4 participants