You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Should include three text description boxes (and new fields on the Link model):
Performance rationale: 8192 char limit
Criticality rationale: 8192 char limit
Gaps: 8192 char limit
Also:
Variable/attribute: TBD: what is this?
Rated by: The person who rated the link. TBD: How to determine this value? The first person to apply the rating, or the most recent person to update the rating? Allow the value to be selected from a dropdown? Determine it automatically, but allow admins to override (I would consider this a separate feature; for the purpose of this ticket let's pick only one thing)?
See glossary for more info
The text was updated successfully, but these errors were encountered:
Hey @Sherwin-14 :) This is still relevant. Looks like we currently only have performance and criticality rating, but we need to add rationale and the others:
'performance_rating>0 and performance_rating<101',
name='p1-100',
),
nullable=False,
)
criticality_rating=Column(
Integer,
CheckConstraint(
'criticality_rating>0 and criticality_rating<11',
name='c1-10',
),
nullable=False,
)
I am updating the issue description to include what I know. I'm thinking we could create a constant (if there isn't already one) so we can update all the large text fields at once, e.g. DB_TEXT_SIZE_LARGE = 8196, and similar constants for the other common string lengths we use. That way it will be easier to change them if we get them wrong :)
@hazelshapiro I'm adding some "TBD" markers to the issue description. Can you fill those in and also verify the character limits?
Should include three text description boxes (and new fields on the
Link
model):Also:
See glossary for more info
The text was updated successfully, but these errors were encountered: