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: added projectDeepLink to template variables #165

Merged
merged 2 commits into from
Mar 3, 2022

Conversation

giautm
Copy link
Contributor

@giautm giautm commented Feb 27, 2022

This PR added a new variable: projectDeepLink to support open Expo Go when the user using the Github Mobile app.

Copy link
Member

@byCedric byCedric left a comment

Choose a reason for hiding this comment

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

Great idea! Left a comment related to the URL and a test.

I'm also a bit mixed about the DeepLink naming, technically we have two types of deeplinks. Right now, we only support Expo Go/managed links but we definitely want to support dev client links. For now, I think this is ok, since it depends on the project structure itself (which we can detect). But maybe you have some thoughts on that? 😄

src/expo.ts Outdated Show resolved Hide resolved
preview-comment/README.md Outdated Show resolved Hide resolved
src/expo.ts Outdated Show resolved Hide resolved
@byCedric
Copy link
Member

byCedric commented Mar 3, 2022

Also, quick sidenote, the failing CI tests can be ignored. If you rebase on main, they should not fail again.

@giautm
Copy link
Contributor Author

giautm commented Mar 3, 2022

Hey @byCedric, can you approve the workflow again for testing. Thank you. ❤️

@byCedric
Copy link
Member

byCedric commented Mar 3, 2022

Hmm, seems like I messed up some of the workflow conditions. Let's leave CI for what it is and just merge it! 😄

@byCedric byCedric merged commit 2603f85 into expo:main Mar 3, 2022
@byCedric
Copy link
Member

byCedric commented Mar 3, 2022

Thanks, I'll release it tomorrow

@giautm giautm deleted the project-deep-link branch March 4, 2022 06:46
github-actions bot pushed a commit that referenced this pull request Mar 4, 2022
## [7.1.0](7.0.3...7.1.0) (2022-03-04)

### New features

* add projectDeepLink to template variables ([#165](#165)) ([2603f85](2603f85))

### Other chores

* run authed tests only when auth is available ([5815161](5815161))
* set major tag version to release commit ([b516013](b516013))
* skip steps if auth is unavailable ([bb7c723](bb7c723))
* update github action author name ([#166](#166)) ([520d959](520d959))
@github-actions
Copy link

github-actions bot commented Mar 4, 2022

🎉 This PR is included in version 7.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants