Skip to content

Add assignment time lobby option#28

Open
JeroenDeDauw wants to merge 1 commit into
FAForever:developfrom
JeroenDeDauw:add-assign-time-option
Open

Add assignment time lobby option#28
JeroenDeDauw wants to merge 1 commit into
FAForever:developfrom
JeroenDeDauw:add-assign-time-option

Conversation

@JeroenDeDauw

Copy link
Copy Markdown
Contributor

This allows hosts to change the assignment time via lobby options.

In some cases, such using prebuild or on as small maps, it does not make sense to wait
a whole 8 minutes for the assignment. The new option allows for much quicker assignment
without changing the default.

This allows hosts to change the assignment time via lobby options.

In some cases, such using prebuild or on as small maps, it does not make sense to wait
a whole 8 minutes for the assignment. The new option allows for much quicker assignment
without changing the default.
@JeroenDeDauw JeroenDeDauw force-pushed the add-assign-time-option branch from 8795010 to edb2d32 Compare December 22, 2020 18:09

@1-alex98 1-alex98 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks solid

@1-alex98

Copy link
Copy Markdown
Member

@JeroenDeDauw I think there is not much activity on Phantom mod. Maybe you want to join your Zulip(chat service) we can send you an invite

@JeroenDeDauw

Copy link
Copy Markdown
Contributor Author

@axel1200 thanks. Maybe you can link these two PR there if that is what is needed?

@1-alex98

Copy link
Copy Markdown
Member

They have been linked but nobody feels responsible I guess.

@JeroenDeDauw

Copy link
Copy Markdown
Contributor Author

@aeoncleanse

@Spikey84

Spikey84 commented Feb 4, 2023

Copy link
Copy Markdown
Collaborator

The timer does not reflect the option selected. Not sure if the author is still active, if not i will deal with this later.

@Spikey84 Spikey84 self-requested a review February 4, 2023 14:44
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.

3 participants