In this PR, I improved the banner message by replacing the userid and commonsid to their actual names.
Screenshots
Before, it was showing the number only, which can be confusing for users.
Now, it shows the full name and commons name.
also removed the welcoming name in announcement as it might be confusing.
(visiting Harold's play page in this case)
Tests
[ ] Backend Unit tests (mvn test) pass
[ ] Backend Test coverage (mvn test jacoco:report) 100%
[ ] Backend Mutation tests (mvn test pitest:mutationCoverage) 100%
[ ] Frontend Unit tests (npm test) pass
[ ] Frontend Test coverage (npm run coverage) 100%
Overview
In this PR, I improved the banner message by replacing the userid and commonsid to their actual names.
Screenshots
Before, it was showing the number only, which can be confusing for users.
Now, it shows the full name and commons name. also removed the welcoming name in announcement as it might be confusing. (visiting Harold's play page in this case)
Tests
mvn test
) passmvn test jacoco:report
) 100%mvn test pitest:mutationCoverage
) 100%npm test
) passnpm run coverage
) 100%npx stryker run
) 100%npx eslint --fix src
)Linked Issues
Closes #0