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

Problem report should still show the name of a deleted user #347

Open
chrisjwelly opened this issue Aug 9, 2019 · 0 comments
Open

Problem report should still show the name of a deleted user #347

chrisjwelly opened this issue Aug 9, 2019 · 0 comments

Comments

@chrisjwelly
Copy link
Member

In #346 , I have added a placeholder "(Deleted member)" if a problem report is made by a user which has been deleted from the database.

Ideally, even if the user has been deleted, we should still be able to show the user's name (perhaps with a tag that he or she is an old member so as not to confuse the new member).

A quick dirty fix is to save the reporter's name in the Problem Report model, but this may not be ideal as there will be a change in the database..

Not critical as this problem will only occur for a new academic year. We should also at least add tests for this.

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

No branches or pull requests

1 participant