Skip to content

Changed wording of Manage Proposals to My Proposals if the user is a student.#1762

Open
ACBerea wants to merge 3 commits into
developmentfrom
manageProposeal_ad
Open

Changed wording of Manage Proposals to My Proposals if the user is a student.#1762
ACBerea wants to merge 3 commits into
developmentfrom
manageProposeal_ad

Conversation

@ACBerea

@ACBerea ACBerea commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Issue Description

Fixes issue #1749

  • Student navigating in their CCE Minor Profile should have My Proposals instead of Manage Proposal within the two tabs. This is to improve clarity. Wording for admin can remain unchanged.

Changes

  • Added Jinja code in the profile.html Manage Proposals tab that changes the text to "My Proposals" if the user is a student. Otherwise, the text will read "Manage Proposals" (it should remain this way for admins).

Testing

  • To test for students:
    • In My Profile, click on the Community & Civic Engagement Minor tab.
    • Toggle on the "Are you interested in the CCE Minor?".
    • Reload the page.
    • Click the newly appeared Manage CCE Minor button next to the View Service Transcript button.
    • Check the second tab to see if it says My Proposals.
  • To test for Admin:
    • Click on the Admin button on the sidebar
    • Select Minor Management
    • Click on the name of a student highlighted in blue.
    • On the new page, check to see if the second tab says Manage Proposals.

Copilot AI review requested due to automatic review settings July 16, 2026 18:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the CCE Minor Profile tab label to improve student-facing clarity while preserving the existing admin wording, aligning with issue #1749.

Changes:

  • Replaced the static “Manage Proposals” tab text with a Jinja conditional that shows “My Proposals” for students and “Manage Proposals” otherwise.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ACBerea
ACBerea requested a review from fritzj2 July 16, 2026 18:24
Comment thread app/templates/minor/profile.html Outdated
…My Proposals to not change to Manage Proposals when on an account that wasn't the displayed student.
Copilot AI review requested due to automatic review settings July 16, 2026 19:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Comment thread app/templates/minor/profile.html Outdated
Copilot AI review requested due to automatic review settings July 16, 2026 19:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread app/templates/minor/profile.html

@fritzj2 fritzj2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

@MImran2002 MImran2002 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good

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.

4 participants