Skip to content

Update lifecycle-status.md#3152

Draft
amycole501 wants to merge 3 commits into
mainfrom
amycole501-add-proposal-status
Draft

Update lifecycle-status.md#3152
amycole501 wants to merge 3 commits into
mainfrom
amycole501-add-proposal-status

Conversation

@amycole501
Copy link
Copy Markdown
Contributor

@amycole501 amycole501 commented Apr 4, 2025

Added new proposal status for "Converting to pattern". Updated both mobile and desktop views.

Added new proposal status for converting to pattern
@juliasolorzano juliasolorzano self-requested a review April 4, 2025 14:54
@juliasolorzano
Copy link
Copy Markdown

@amycole501 adding the preview url for your reference.

Comment thread pages/ui-components/lifecycle-status.md Outdated
Co-authored-by: Julia Solórzano <julia.solorzano@gsa.gov>
Copy link
Copy Markdown

@juliasolorzano juliasolorzano left a comment

Choose a reason for hiding this comment

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

@amycole501 added a few suggestions for your review.

I believe you will also need to add this data to the following files:

{% assign status_classes = rejected_classes %}
{% elsif item.status_minor == "accepted" %}
{% assign status_classes = accepted_classes %}
{% elsif item.status_minor == "converting to pattern" %}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You need to add an assignment below this elsif statement in order for your new classname to apply. Indent two spaces and add the follow:

{% assign status_classes = lifecycle-bg--convert %}

{% assign status_classes = rejected_classes %}
{% elsif item.status_minor == "accepted" %}
{% assign status_classes = accepted_classes %}
{% elsif item.status_minor == "converting to pattern" %}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same suggestion as above:

{% assign status_classes = lifecycle-bg--convert %}

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.

3 participants