Skip to content

Conversation

@dwoodruffsf
Copy link
Contributor

Merges release/v0.1.0-experimental back to develop. Things to know:

  • ui_atlas.png was deleted in dev and modified in release. I kept the modified version.
  • MainMenu had lighting changes that conflicted, with release version taken.
  • UnityLogo.prefab anchor position conflicted--release version taken.
  • PostGame had a GameObject that was modified in dev and completely removed in release. I removed it.

I tested by playing through a couple losses, and then playing through a victory with one connected client. Pretty much all worked the way I expected.

There is no novel code in this change, so we will be taking it directly.

LukeStampfli and others added 30 commits March 8, 2021 15:36
Changed the URL to the readme as the current version generates a broken link.
#
#
…aliasing

Enabled antialiasing in all of the relevant scene cameras (SMAA)
Removes earlier temp code that hard-coded separate sprite lists per class. Now uses the Sprites assigned in the GameDataSource. If a class has an unimplemented skill (or the skill has no assigned Sprite), the button for that skill is hidden.
…erimental-status-in-readme

Highlight EARLY ACCESS status in readme
…for_unimplemented_attacks

Bug fix: hide icons for unimplemented attacks
LukeStampfli and others added 20 commits March 30, 2021 10:26
* release/v0.1.0-experimental: (97 commits)
  adding screenshots (#228)
  [GOMPS-352] Add audiomixer to project (#222)
  feature: transport picker improvements (#200)
  fixing issue where client disconnect boots everyone else (#229)
  removed GC allocs inside of AIBrain (#210)
  init client door with server's current value for IsOpen (#227)
  Photon Realtime Setup Guide for Boss Room (#225)
  Update README.md add link to doc (#226)
  Bug fix: hide icons for unimplemented attacks
  feature: v0.1.0 experimental updating to latest master mlapi (#220)
  fixing missing reference exceptions, and missing animation (#219)
  feat: replace with photon package (#218)
  Replaced LobbyPlayerArray with a NetworkList<LobbyPlayerState> (#217)
  ART:  Warning Message fixes for models, animations, and Shader...and new App icon (#213)
  Updating to latest MLAPI on master (#214)
  null guard added inside of target action for accessing a possibly destroyed reticule (#212)
  [GOMPS-270] bossroom should let users specify connection port# (#205)
  Fix useless log error (#203)
  Bugfix: properly disconnect on quit [GOMPS-324][GOMPS-327][GOMPS-385] (#196)
  Volume control [GOMPS-373] (#195)
  ...

# Conflicts:
#	README.md
…hat was outside of BossRoom folder and placed textures in VFX/Textures folder (#239)
FPaizSF
FPaizSF previously approved these changes Apr 5, 2021
Copy link
Contributor

@FPaizSF FPaizSF 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!

FPaizSF
FPaizSF previously approved these changes Apr 5, 2021
In release branch, it was using a sprite in ui_atlas (specifically "UI_elements_atlas_hud_0"). I think it just got missed when the close button was broken out to a separate image. I plugged in the new ui_btn_exit sprite.
Copy link
Contributor

@eheimburg eheimburg left a comment

Choose a reason for hiding this comment

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

  • Fixed a small prefab error (that ui_atlas was still being used in one place!)
  • Noticed that the boss can't break the pillars at all. But this is actually not due to the merge, since it's broken in the release branch also! Somewhere along the line, my boss-breakable prefab got wiped out. Will revise the pillar prefab again in develop post-merge.

@dwoodruffsf dwoodruffsf merged commit c8209d0 into develop Apr 6, 2021
@SamuelBellomo SamuelBellomo deleted the release_merge_april_v2 branch April 6, 2021 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.