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.

Sign up for GitHub

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

Jump to bottom

Remove IDs from the Video Screen and Add Course Name and Chapter Name instead #154

Closed
SudharakaP opened this issue Oct 20, 2019 · 4 comments · Fixed by #161
Closed

Remove IDs from the Video Screen and Add Course Name and Chapter Name instead #154

SudharakaP opened this issue Oct 20, 2019 · 4 comments · Fixed by #161
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed

Comments

@SudharakaP
Copy link
Member

SudharakaP commented Oct 20, 2019 •
edited
Loading

Is your feature request related to a problem? Please describe.

Currently in the Videos entity screen we display chapter and course IDs.

image

So when the admin wants to know which chapter or course a video belongs to, he/she has to do a search on the relevant ID on the Chapter or Course entity screens.

Describe the solution you'd like

Remove the ID columns as they are useless to the end user. Instead add Course Name and Chapter Name columns with hyperlinks to the relevant Course or Chapter itself.
@SudharakaP SudharakaP added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Oct 20, 2019
@SudharakaP SudharakaP changed the title Remove IDs from the Video Screen and Add Course and Chapter Remove IDs from the Video Screen and Add Course Name and Chapter Name instead Oct 20, 2019
@SudharakaP SudharakaP mentioned this issue Oct 20, 2019
Closed
2 tasks
@inspire99
Copy link
Contributor

Hi @SudharakaP , I shall work on this.

Best Regards
Ganesh

@SudharakaP
Copy link
Member Author

@inspire99 : Thanks for the contributions. 👍

inspire99 added a commit to inspire99/OpenLearnr that referenced this issue Oct 27, 2019
@inspire99
Fixes OpenArchitex#154: Remove ID from Video screen, add chaptername …
32a53b8 
…and courseName instead

Fixes OpenArchitex#154: Remove ID from Video screen, add chaptername and courseName instead
@inspire99 inspire99 mentioned this issue Oct 27, 2019
Merged
5 tasks
@inspire99
Copy link
Contributor

Hi @SudharakaP ,
I have created a pull request --> #161

It contains changes to remove ID, and replace with name contents in video screen.

Regards
Ganesh

inspire99 added a commit to inspire99/OpenLearnr that referenced this issue Oct 27, 2019
@inspire99
Fixes OpenArchitex#154: Remove ID from Video screen, add chaptername …
237335b 
…and courseName instead

Fixes OpenArchitex#154: Remove ID from Video screen, add chaptername and courseName instead
SudharakaP added a commit that referenced this issue Oct 27, 2019
@SudharakaP
Merge pull request #161 from inspire99/develop
5d5a409 
Fixes #154: Remove ID from Video screen, add chaptername and courseNa…
@SudharakaP
Copy link
Member Author

@inspire99 : Wonderful. 😄 Thanks much for your time and work 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #154: Remove ID from Video screen, add chaptername and courseNa… inspire99/OpenLearnr
2 participants
@SudharakaP @inspire99