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.

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

feat: integrate video detail API & tags list API #27

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

arbisoft-qaisarirfan
Copy link

@arbisoft-qaisarirfan arbisoft-qaisarirfan commented Feb 13, 2025

PR Description: Integrate Video Detail & Tags List APIs

Summary

This PR integrates the Video Detail API and Tags List API into the project, ensuring video details are fetched dynamically and tags are displayed correctly.

Link to the associated Taiga ticket: https://projects.arbisoft.com/project/arbisoft-sessions-portal-20/us/69


Changes Implemented

Integrated Video Detail API using:

GET /api/v1/events/videoasset/{id}/
  • Fetches video details based on the provided id.
  • Implements loading, success, and error handling.
  • Ensures TypeScript types are correctly defined.

Integrated Tags List API using:

GET /api/v1/events/tags/
  • Fetches and displays the list of available tags.
  • Optimized for performance with caching/memoization where applicable.

Updated UI components to dynamically display video details and tags.

Handled API errors gracefully with proper error messages.


How to Test

1️⃣ Run the application and navigate to a video details page.
2️⃣ Verify that video details are correctly fetched and displayed.
3️⃣ Ensure the tags list is populated from the API.
4️⃣ Test error handling by simulating network failures.


Impact

  • Improves user experience by dynamically fetching video details.
  • Ensures accurate tag data from the API.
  • Enhances maintainability with proper API handling and TypeScript typing.

🔗 Reviewers: (@sameeramin @farhan2742 @arslanather @wkhaliddev1)

Screen.Recording.2025-02-13.at.1.04.36.PM.mov

screencapture-localhost-3000-en-videos-1-2025-02-13-13_05_23

Ready for review! 🚀

@@ -1,11 +1,11 @@
{
"name": "reactjs-boilerplate",
"name": "session-portal",

Choose a reason for hiding this comment

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

I have seen a similar change in another of your PR!

<Typography variant="h3" color="primary" width="100%">
Videos
</Typography>
<Link href="/videos/asdf">Video Detail</Link>

Choose a reason for hiding this comment

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

Shouldn't this be coming from language.json?

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.

2 participants