This repository has been archived by the owner on Jan 18, 2024. It is now read-only.
Add originalFullName to the public config
#3494
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
brentvatne
approved these changes
May 18, 2021
This was referenced May 20, 2021
ide
reviewed
May 20, 2021
| @@ -143,9 +143,11 @@ export function getConfig(projectRoot: string, options: GetConfigOptions = {}): | |||
| delete configWithDefaultValues.exp.android.config; | |||
| } | |||
|
|
|||
| // This value will be overwritten when the manifest is being served from the host (i.e. not completely accurate). | |||
| // These value will be overwritten when the manifest is being served from the host (i.e. not completely accurate). | |||
Merged
3 tasks
wschurman
added a commit
to expo/expo
that referenced
this pull request
Feb 6, 2023
…rted config (#21070) # Why This reverts expo/expo-cli#3376 and expo/expo-cli#3494. These were added for AuthSession and Push notification registration. Neither of these use this anymore: - AuthSession deprecated use of the proxy use with the field from the manifest: #17327. It supports supplying them via arguments. - Push notification prefers `projectId` now: #14265. Also it allows supplying them via arguments. These auto config augmentations are causing issues with `eas build` since the generated full name could differ from the actual full name: 1. Create project in an org on the website. 2. `npx expo init ...` 3. `eas init --id <>` 4. Remove `owner` from manifest. 5. `eas build` 6. Download artifact, inspect embedded config, see that it has the correct `projectId` yet also a `currentFullName` and `originalFullName` containing the current logged-in user rather than the actual one corresponding to the projectId. # How One fix we could do here would be to read the projectId and get the truthful full name, but that adds a network dependency and is probably not worth it. So, instead, I decided to just remove the hack since they are no longer needed in the two libraries this was added for. # Test Plan Run all tests. # Checklist <!-- Please check the appropriate items below if they apply to your diff. This is required for changes to Expo modules. --> - [ ] Documentation is up to date to reflect these changes (eg: https://docs.expo.dev and README.md). - [ ] Conforms with the [Documentation Writing Style Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md) - [ ] This diff will work correctly for `expo prebuild` & EAS Build (eg: updated a module plugin).
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why
#3376 added
currentFullNamefor use as a stable identifier, but it is not stable through project transfers.originalFullNameis, so it should be used instead in libraries like auth-session and notifications.How
currentFullNameto the public config #3376 but fororiginalFullName.currentFullNameis kept so that the CLI continues to function until we change the auth-session and notifications libraries to useoriginalFullName.originalFullNamesimilar to [notifications] Add support for custom managed workflow expo#12465 and [auth session] Add support for custom managed workflow expo#12464.Test Plan
Run same unit tests that were added in #3376.
In a sample project run:
~/expo/expo-cli/node_modules/.bin/expo config --type public, see bothcurrentFullNameandoriginalFullNamein output.