Fix not found 404 when route to session from speaker view

[odkhang]

Summary by Sourcery

Fix the 404 error when routing to a session from the speaker view by updating the session link URL to include the correct base path.

Bug Fixes:

• Fix the 404 error when navigating to a session from the speaker view by ensuring the correct base URL is used in the session link.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request fixes a 404 "Not Found" error that occurred when routing to a session from the speaker view. The change modifies the URL generation for session links by incorporating the BASE_URL environment variable, ensuring proper routing in different deployment environments.

File-Level Changes

Change	Details	Files
Modified URL generation for session links	Added process.env.BASE_URL to handle different deployment environments Introduced a fallback value of '/video/' if BASE_URL is not set Updated the return statement to use template literals for string interpolation	webapp/src/views/schedule/speakers/item.vue

Sequence Diagram

No sequence diagram generated.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @odkhang - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider making the default path ('/video/') configurable or environment-specific to ensure flexibility across different deployment scenarios.

Here's what I looked at during the review

• 🟡 General issues: 1 issue found
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[sourcery-ai]

issue (bug_risk): Consider potential issues with dynamic base URL in generateSessionLinkUrl

The change introduces a dynamic base URL, which could lead to inconsistent URL generation. Consider the following:

1. Is process.env.BASE_URL guaranteed to be set in all environments?
2. The fallback value '/video/' seems arbitrary. Is this intentional?
3. There's a potential for double slashes in the URL if BASE_URL ends with a slash.