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

[Merged by Bors] - Use bevy with default features in iOS example #7042

Closed
wants to merge 1 commit into from
Closed

[Merged by Bors] - Use bevy with default features in iOS example #7042

wants to merge 1 commit into from

Conversation

silvestrpredko
Copy link
Contributor

@silvestrpredko silvestrpredko commented Dec 27, 2022

Objective

I am new to Bevy. And during my development, I noticed that the iOS example doesn't work.
Example panics with next message: panicked at 'Resource requested by bevy_ui::widget::text::text_system does not exist: bevy_asset::assets::Assets.

I have asked for help in a discord iOS chat and there I receive a recommendation that it is possible that some bevy features missing.

Solution

So, I used bevy with default features.

@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior C-Examples An addition or correction to our examples labels Dec 27, 2022
@alice-i-cecile
Copy link
Member

Thanks for testing and the fix!

@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Dec 27, 2022
Copy link
Member

@mockersf mockersf left a comment

Choose a reason for hiding this comment

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

I don't think there's a reason anymore to just enable select features on iOS.

Could you enable all default features using

bevy = { path = "../../" }

@alice-i-cecile alice-i-cecile removed the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Dec 28, 2022
	* It's fixes a bug related to a missing ```asset``` in a ```text_system```
@alice-i-cecile
Copy link
Member

Awesome, if this works can you update the PR description and title?

bors try

bors bot added a commit that referenced this pull request Dec 28, 2022
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Dec 28, 2022
@alice-i-cecile
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Dec 28, 2022
# Objective

I am new to Bevy. And during my development, I noticed that the `iOS` example doesn't work.
Example panics with next message: ```panicked at 'Resource requested by bevy_ui::widget::text::text_system does not exist: bevy_asset::assets::Assets```.

I have asked for help in a `discord` iOS chat and there I receive a recommendation that it is possible that some bevy features missing.

## Solution

So, I added a ```bevy_sprite``` and it fixes the problem :)
@alice-i-cecile
Copy link
Member

bors r-

@bors
Copy link
Contributor

bors bot commented Dec 28, 2022

Canceled.

@alice-i-cecile
Copy link
Member

Waiting a bit so @silvestrpredko has a chance to update the PR title :)

@silvestrpredko silvestrpredko changed the title Add to bevy feature bevy_sprite in iOS example Use bevy with default features in iOS example Dec 28, 2022
@silvestrpredko
Copy link
Contributor Author

Waiting a bit so @silvestrpredko has a chance to update the PR title :)

Done, sorry for the delay. I miss updating a PR title.

@mockersf
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Dec 28, 2022
# Objective

I am new to Bevy. And during my development, I noticed that the `iOS` example doesn't work.
Example panics with next message: ```panicked at 'Resource requested by bevy_ui::widget::text::text_system does not exist: bevy_asset::assets::Assets```.

I have asked for help in a `discord` iOS chat and there I receive a recommendation that it is possible that some bevy features missing.

## Solution

So, I used ```bevy``` with default features.
@bors bors bot changed the title Use bevy with default features in iOS example [Merged by Bors] - Use bevy with default features in iOS example Dec 28, 2022
@bors bors bot closed this Dec 28, 2022
alradish pushed a commit to alradish/bevy that referenced this pull request Jan 22, 2023
# Objective

I am new to Bevy. And during my development, I noticed that the `iOS` example doesn't work.
Example panics with next message: ```panicked at 'Resource requested by bevy_ui::widget::text::text_system does not exist: bevy_asset::assets::Assets```.

I have asked for help in a `discord` iOS chat and there I receive a recommendation that it is possible that some bevy features missing.

## Solution

So, I used ```bevy``` with default features.
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

I am new to Bevy. And during my development, I noticed that the `iOS` example doesn't work.
Example panics with next message: ```panicked at 'Resource requested by bevy_ui::widget::text::text_system does not exist: bevy_asset::assets::Assets```.

I have asked for help in a `discord` iOS chat and there I receive a recommendation that it is possible that some bevy features missing.

## Solution

So, I used ```bevy``` with default features.
@silvestrpredko silvestrpredko deleted the silvestr/fix-ios-example branch September 14, 2023 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Bug An unexpected or incorrect behavior C-Examples An addition or correction to our examples S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants