Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

Commit

Permalink
Do not output project URL in bare, it is irrelevant
Browse files Browse the repository at this point in the history
  • Loading branch information
brentvatne committed Apr 11, 2020
1 parent 117d726 commit ccf7007
Showing 1 changed file with 17 additions and 10 deletions.
27 changes: 17 additions & 10 deletions packages/expo-cli/src/commands/publish.ts
Original file line number Diff line number Diff line change
Expand Up @@ -160,17 +160,24 @@ export async function action(projectDir: string, options: Options = {}) {
);
}

// TODO: replace with websiteUrl from server when it is available, if that makes sense.
let websiteUrl = url.replace('exp.host', 'expo.io');
log(
`The project page is: ${terminalLink(websiteUrl, websiteUrl)}. ${terminalLink(
'Learn more.',
'https://expo.fyi/project-page'
)}`
);
if (target === 'managed') {
// TODO: replace with websiteUrl from server when it is available, if that makes sense.
let websiteUrl = url.replace('exp.host', 'expo.io');
log(
`The project page is: ${terminalLink(websiteUrl, websiteUrl)}. ${terminalLink(
'Learn more.',
'https://expo.fyi/project-page'
)}`
);

if (recipient) {
await sendTo.sendUrlAsync(websiteUrl, recipient);
if (recipient) {
await sendTo.sendUrlAsync(websiteUrl, recipient);
}
} else {
// This seems pointless in bare?? Leaving it out
// if (recipient) {
// await sendTo.sendUrlAsync(url, recipient);
// }
}
} finally {
if (startedOurOwn) {
Expand Down

0 comments on commit ccf7007

Please sign in to comment.