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.

Sign up for GitHub

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

Jump to bottom

Build module refactor #492

Merged
merged 2 commits into from
Sep 9, 2018
Merged

Build module refactor #492

merged 2 commits into from
Sep 9, 2018

Conversation

veteran29
Copy link
Member

@veteran29 veteran29 commented Sep 9, 2018 •
edited
Loading

Q A
Bug fix? no
New feature? no
Needs wipe? no
Fixed issues none

Description:

Refactoring of the whole build module into more flexible implementation.
Does not fix #470 as there are still some features missing, this issue needs to be moved to next sprint.

Content:

  • Refactored build module
  • Building of objects on mouse position
  • Build multiple items with ctrl

Tested on:

  • Local MP Vanilla
  • Local MP ACE
  • Dedicated MP Vanilla
  • Dedicated MP ACE

@veteran29 veteran29 added this to the Sprint 5 milestone Sep 9, 2018
@veteran29 veteran29 requested a review from Wyqer September 9, 2018 20:15
@veteran29 veteran29 merged commit 928cc17 into v0.97S5 Sep 9, 2018
@Wyqer Wyqer added done and removed review/QA labels Sep 9, 2018
@veteran29 veteran29 deleted the 0.97S5-470 branch September 9, 2018 23:27
@Wyqer Wyqer mentioned this pull request Sep 10, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wyqer Wyqer Wyqer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Sprint 5
Development

Successfully merging this pull request may close these issues.
2 participants
@veteran29 @Wyqer