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

Added new gameplay video options #1314

Merged
merged 6 commits into from
Dec 10, 2022
Merged

Added new gameplay video options #1314

merged 6 commits into from
Dec 10, 2022

Conversation

whyisthesheep
Copy link
Contributor

@whyisthesheep whyisthesheep commented Oct 22, 2022

Description

Added:

  • minecraft-2
  • multiversus
  • fall-guys
  • steep

as options for background gameplay

Issue Fixes

None

Checklist:

  • I am pushing changes to the develop branch
  • I am using the recommended development environment
  • I have performed a self-review of my own code
  • My changes generate no new warnings
  • My changes follow the existing code-style
  • My changes are relevant to the project

Any other information (e.g how to test the changes)

You can try setting the background video to

  • minecraft-2
  • multiversus
  • fall-guys
  • steep

Added:
- minecraft-2
- multiversus
- fall-guys
- steep
as options for background gameplay
Added:
- minecraft-2
- multiversus
- fall-guys
- steep
as options for background gameplay

also CONSTANTS.py didn't seem to exist in the develop branch
@whyisthesheep whyisthesheep marked this pull request as draft October 22, 2022 22:31
@whyisthesheep whyisthesheep marked this pull request as ready for review October 22, 2022 22:31
@github-actions github-actions bot added the stale label Oct 29, 2022
@OpenSourceSimon OpenSourceSimon added keep and removed stale labels Nov 7, 2022
@JasonLovesDoggo
Copy link
Collaborator

Do you have permission from the video creator?

@github-actions github-actions bot added stale and removed stale labels Nov 15, 2022
`fall-guys` and `multiversus` where changed
@whyisthesheep
Copy link
Contributor Author

Do you have permission from the video creator?

Hi I have just checked over and replaced 2 of the videos and now all of the videos which I have commited are "free to use" (from video description on youtube)

Copy link
Collaborator

@OpenSourceSimon OpenSourceSimon left a comment

Choose a reason for hiding this comment

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

Thanks for your PR! Although, we switched to a utils/backgrounds.json file instead of CONSTANTS.py in the develop branch. Can you adapt the changes to that?

Deleted for backgrounds.json
Just took the new entries from the old `CONSTANTS.py` and moved them here (and changed the format so that it works!)
(Removed something that was commented out (in python)
@whyisthesheep
Copy link
Contributor Author

Thanks for your PR! Although, we switched to a utils/backgrounds.json file instead of CONSTANTS.py in the develop branch. Can you adapt the changes to that?

Hi, my bad for not doing that it should be sorted now.

Copy link
Collaborator

@OpenSourceSimon OpenSourceSimon left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@JasonLovesDoggo
Copy link
Collaborator

Looks good to me as well, I will merge tonight

@github-actions github-actions bot added stale and removed stale labels Nov 24, 2022
@github-actions github-actions bot added stale and removed stale labels Dec 6, 2022
@OpenSourceSimon OpenSourceSimon merged commit 85d0f33 into elebumm:develop Dec 10, 2022
@whyisthesheep whyisthesheep deleted the develop branch December 14, 2022 01:11
@whyisthesheep whyisthesheep restored the develop branch December 14, 2022 01:11
@whyisthesheep whyisthesheep deleted the develop branch December 14, 2022 01:11
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.

3 participants