-
-
Notifications
You must be signed in to change notification settings - Fork 395
expanded AddPitchLit to ski #6576
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
Conversation
|
Uhh... I recognize there has been some prior discussion to this, but I don't see anything conclusive there. Regarding the implementation:
Regarding asking for it in the first place: Does it fulfill all the quest guidelines? Can you explain it using the quest template? |
Aye, see #6518 (comment) |
So? Doesn't make sense to me. Is the light emitted by piste lighting only visible to those who ski? |
Note the "and". So I guess the useful purpose is that it can also be used after dark? Then again, don't pistes have opening hours? |
It is very useful. And imho correct to be
Pistes ways are not tagged with that. Note: I am skiing every winter for one week since 2008. |
No, but it often does not operate outside of skiing season, or on certain special days. Thus it is less relevant for hikers or other users in the area. |
In addition to what @HolgerJeromin said: this often depends on the type of piste. nordic pistes rarely have any form of time restriction in my experience, as long as there is sufficient snow. |
|
But perhaps the whole thing is better be done as a separate quest as in SCEE. |
Ok, I understand. |
|
Last point: Do you think it makes sense to ask again every X years, @HolgerJeromin ? |
As this is quite complicated to add I would go for 6 years or later (or never). |
|
I don't care either way, your call, @paulklie |
I don't think its necessary:
|
Fixes #6518.
Instead of implementing this as a new quest, I suggest we expand the existing IsPitchLit quest to ski pistes.
The string "Is this lit here?" already fits well.
It does not ask for piste routes, since these are filtered out (for performance reasons) anyway.