Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Allow future discontinued date #2204
Allow future discontinued date #2204
Changes from 23 commits
c6e136b
8e200be
d5e1608
8c173c3
7edc4c2
c88549b
1c7e0e1
1c27047
6357e7f
91ad6eb
a037fd5
b2557c1
255bce2
e8edabb
f23b55b
8cbc146
f72e721
b394331
555e1a6
b9e984f
ae807e8
24214f7
f54af1c
82b206a
b5684a6
ffd151d
b354751
ccf94f1
d30fbd2
ef0510a
a3d8694
368df56
427ad2b
34382bd
890f226
7ae9295
bf63c0f
6139624
4a73fde
4b37398
7341677
edb84fe
ea294dc
7259212
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we invent a new classification status, we need to think about how this is displayed in the notifications search UI. It will need an icon, or at least to check the edges code that shows the notifications search that it falls back to the text gracefully
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you suggest using one of the existing ones?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, I think a new one makes sense, though "status" is a bit of an overloaded term. Perhaps "alert"? Then the icon can be a bell or something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To do this, set the status in the constants like you have done, for this new "alert", then in
portality/static/js/edges/notifications.edge.js
add a new SVG icon in theicons
dictionary, and a new description in theclassifications
dictionary and this should then display as appropriateThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implemented but then appeared not to be needed. Is it worth leaving or should be scrapped? Not currently used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest to scrap if not used