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

fix sorting of the manually linked files #1061

Closed
wants to merge 1 commit into from

Conversation

revam
Copy link
Member

@revam revam commented Sep 9, 2024

Fix sorting of the manually linked files by episode type, episode number, followed by file creation date.

If the manually linked components were refactored to use the episode endpoint first and foremost then we might have avoided to do this client-side sorting. But. I'm not going to put in the effort to refactor them all at this point in time, so this band-aid will do for now. At least until somebody puts in the time and effort to do the work.

by episode type, episode number, followed by file creation date.

If the manually linked components were refactored to use the episode endpoint first and foremost then we might have avoided to do this client-side sorting. But. I'm not going to put in the effort to refactor them all at this point in time, so this band-aid will do for now. At least until somebody puts in the time and effort to do the work.
@harshithmohan
Copy link
Member

The UI for this page and will change to the dual pane one like release management and then the episode list will be paginated, and then we cannot sort it. What we need to do is to get the episodes instead of files. Is that possible? 🤔

@harshithmohan
Copy link
Member

Not needed due to #1067

@harshithmohan harshithmohan deleted the fix-manually-linking-sort branch September 10, 2024 18:40
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