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 to Projectionist Profiles from Standard List #558

Merged
merged 16 commits into from
Oct 25, 2021

Conversation

Muirrum
Copy link
Member

@Muirrum Muirrum commented Oct 25, 2021

Closes #557

Muirrum and others added 16 commits September 15, 2021 21:00
Fixes breaking readthedocs builds due to removal of Python 2 support in setuptools 58
- Resolves an issue where ticket comments posted in Slack were not being sent to RT
- Resolves a key error that occurred when parsing ticket comment data from the Home tab
Bumps [sqlparse](https://github.com/andialbrecht/sqlparse) from 0.4.1 to 0.4.2.
- [Release notes](https://github.com/andialbrecht/sqlparse/releases)
- [Changelog](https://github.com/andialbrecht/sqlparse/blob/master/CHANGELOG)
- [Commits](andialbrecht/sqlparse@0.4.1...0.4.2)

---
updated-dependencies:
- dependency-name: sqlparse
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
This presents an initial solution to WPI-LNL#432, by adding the text
"[TEST EVENT]" to the end of every event marked as a Test Event in the
database.

Also considered was adding the unicode "⚠" character with a tooltip.
This was not added due to the potential for misinterpreting the symbol
and for accessibility on mobile.
If the user is not able to edit PITs, the list will link to the projectionist's profile.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.007%) to 93.296% when pulling 1bbd89e on Muirrum:link-projectionist-profiles into 136ed7f on WPI-LNL:master.

Copy link
Member

@tnurse18 tnurse18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea!

@tnurse18 tnurse18 merged commit dc40a81 into WPI-LNL:master Oct 25, 2021
tnurse18 added a commit that referenced this pull request Oct 25, 2021
#558 used the wrong user id for the link. This corrects that error.
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.

Link to User Profile from Projectionist List
5 participants