Skip to content
This repository was archived by the owner on Apr 29, 2022. It is now read-only.

Commit 5b7728b

Browse files
MisterRiosmatrixise
authored andcommitted
Cherry-pick PR 1316
1 parent 412bfc7 commit 5b7728b

File tree

2 files changed

+26
-2
lines changed

2 files changed

+26
-2
lines changed

conference/models.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -743,7 +743,10 @@ def get_schedule_url(self):
743743
if not event:
744744
return None
745745

746-
return event.schedule.get_absolute_url() + '?' + urlencode({'selected': self.slug})
746+
url = event.schedule.get_absolute_url()
747+
slug = urlencode({'selected': self.slug})
748+
time = event.start_time.strftime('%H:%M-UTC')
749+
return f"{url}?{slug}#{time}"
747750

748751
def get_slides_url(self):
749752

tests/test_talks.py

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,13 @@
77
from django.utils import timezone
88

99
from conference.models import TALK_STATUS, TALK_LEVEL
10-
from tests.factories import UserFactory, TalkFactory, ConferenceTagFactory, TalkSpeakerFactory
10+
from tests.factories import (
11+
EventFactory,
12+
UserFactory,
13+
TalkFactory,
14+
ConferenceTagFactory,
15+
TalkSpeakerFactory,
16+
)
1117
from tests.common_tools import get_default_conference, redirects_to, template_used, make_user
1218

1319
pytestmark = [pytest.mark.django_db]
@@ -279,3 +285,18 @@ def test_talk_for_other_than_current_conference(client):
279285
resp = client.get(url)
280286

281287
assert resp.status_code == 404
288+
289+
290+
def test_show_talk_link_in_schedule(client):
291+
"""
292+
The talk url points to the schedule, with correct talk slug, and time in utc
293+
"""
294+
get_default_conference()
295+
talk = TalkFactory(status=TALK_STATUS.accepted)
296+
event = EventFactory(talk=talk)
297+
url = talk.get_absolute_url()
298+
299+
response = client.get(url)
300+
301+
start_time = event.start_time.strftime('%H:%M-UTC')
302+
assert f"{talk.slug}#{start_time}" in response.content.decode()

0 commit comments

Comments
 (0)