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

Link username to user tests page from tests_view pages #788

Merged
merged 1 commit into from
Sep 6, 2020

Conversation

linrock
Copy link
Contributor

@linrock linrock commented Sep 6, 2020

Fixes #625 (comment) as suggested by @silversolver1

I mean on the page of an actual test where a user can access the stop/purge, reschedule or modify buttons, next to where the value for itp is listed, there is the word "username". here, the user's particular name does not currently link to that user's /tests/user/ page, which is the feature I am suggesting

I agree this is more practical since I've also commonly wanted to go see a list of the user's tests while on an individual test page.


Approvers will see a link to the user admin page in addition to the link to the user tests page.

image

@ppigazzini ppigazzini added server server side changes enhancement labels Sep 6, 2020
@ppigazzini ppigazzini merged commit aa74b63 into official-stockfish:master Sep 6, 2020
@ppigazzini
Copy link
Collaborator

Tested OK on DEV, PROD updated, thank you @linrock :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement server server side changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ideas for improvement [RFC]
2 participants