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

Project structure #27

Closed
4 of 5 tasks
vaind opened this issue Jun 13, 2022 · 6 comments
Closed
4 of 5 tasks

Project structure #27

vaind opened this issue Jun 13, 2022 · 6 comments
Labels
Feature New feature or request

Comments

@vaind
Copy link
Collaborator

vaind commented Jun 13, 2022

I've had a brief look at this project and tried following the CONTRIBUTING.md to build the project and I have a few suggestions.

Currently, the project is supposed to be checked out into an existing Unreal Engine project as a plugin. This means a developer of sentry-unreal SDK must first create a new UE project (sample), to be able to do any development. This increases the cost-of-entry for new SDK developers. Instead, I'd suggest having a look at https://github.com/getsentry/sentry-unity and think about the following changes:

@vaind vaind added Feature New feature or request Platform: Unreal labels Jun 13, 2022
@vaind
Copy link
Collaborator Author

vaind commented Aug 17, 2022

@tustanivsky I understand you have a demo app to simulate event captures locally? Can you maybe add the app to the sample now that we have moved the plugin to plugin-dev from the repo root?

@tustanivsky
Copy link
Collaborator

@tustanivsky I understand you have a demo app to simulate event captures locally? Can you maybe add the app to the sample now that we have moved the plugin to plugin-dev from the repo root?

Yes, we can do that. Just wanted to clarify - should it include the plugin as well? Or we're going to copy plugin-dev to the <SampleRoot>/Plugins folder in CI i.e. when running integration tests?

@vaind
Copy link
Collaborator Author

vaind commented Aug 17, 2022

it shouldn't include the plugin - ideally, we can create a symlink?

@tustanivsky
Copy link
Collaborator

it shouldn't include the plugin - ideally, we can create a symlink?

I did some quick testing and it looks that UE doesn't recognize symlinks on Windows which is not good considering it's the main platform for Unreal development. On Mac it works just fine though

@vaind
Copy link
Collaborator Author

vaind commented Aug 17, 2022

Which type of link have you tried on Windows? If you could create a PR with the sample as is, I can have a look around and maybe I can think of something.

@tustanivsky
Copy link
Collaborator

Which type of link have you tried on Windows? If you could create a PR with the sample as is, I can have a look around and maybe I can think of something.

I've created a PR (#64) with symlink for Mac. On Windows I've tried to create one with mklink, but when launching the project UE doesn't detect any plugins installed

@vaind vaind closed this as completed Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request
Projects
Archived in project
Development

No branches or pull requests

2 participants