Skip to content

(fix#3132): change titles from Governance Actions to Live Voting#3865

Merged
Ciabas merged 1 commit into
developfrom
3132-title-wrong-on-live-voting-page
Jul 2, 2025
Merged

(fix#3132): change titles from Governance Actions to Live Voting#3865
Ciabas merged 1 commit into
developfrom
3132-title-wrong-on-live-voting-page

Conversation

@Ciabas
Copy link
Copy Markdown
Contributor

@Ciabas Ciabas commented Jul 1, 2025

List of changes

  • Change titles/breadcrumbs/links from Governance Actions to Live Voting

Checklist

  • related issue
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the changelog
  • I have added tests that prove my fix is effective or that my feature works

Changes:

image image image image

@Ciabas Ciabas requested a review from bosko-m July 1, 2025 12:47
@Ciabas Ciabas self-assigned this Jul 1, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

✅ All checks succeeded

@Ciabas Ciabas marked this pull request as ready for review July 1, 2025 12:48
@Ciabas Ciabas requested review from MSzalowski and Ryun1 as code owners July 1, 2025 12:48
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

✅ All checks succeeded

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

✅ All checks succeeded

editDrepMetadata: "/edit_drep",
error: "/error",
faqs: "/faqs",
governanceActions: "/governance_actions",
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.

@Ciabas Cant agree with route changes, unless you tell me the old routes will redirect automatically to new ones?
Some of the GA details were socialzed, shared, so we dont want to "kill" that.
If you cant achieve backward compatibility, then routes should stay the same and just adjust pages titles accordingly.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It was in the task description. I reverted the paths changes.

@Ciabas Ciabas force-pushed the 3132-title-wrong-on-live-voting-page branch from 79bc0b8 to c57ad9f Compare July 1, 2025 14:56
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

✅ All checks succeeded

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

✅ All checks succeeded

export const OUTCOMES_PATHS = {
governanceActionsOutcomes: "/outcomes",
governanceActionOutcomes: "/outcomes/governance_actions/:id",
governanceActionsLiveVoting: "/connected/governance_actions",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I removed that since it a duplicate of other path.
image

@Ciabas Ciabas requested a review from bosko-m July 1, 2025 15:01
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 1, 2025

✅ All checks succeeded

@Ciabas Ciabas merged commit 8abf445 into develop Jul 2, 2025
8 checks passed
@Ciabas Ciabas deleted the 3132-title-wrong-on-live-voting-page branch July 2, 2025 07:28
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 2, 2025

@Ciabas
This PR is in the tag: develop-8abf445e04a9946f00c0e0b1317f4b130c52222e , for govtool-metadata-validation service

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 2, 2025

@Ciabas
This PR is in the tag: develop-8abf445e04a9946f00c0e0b1317f4b130c52222e , for govtool-backend service

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 2, 2025

@Ciabas
This PR is in the tag: develop-8abf445e04a9946f00c0e0b1317f4b130c52222e , for govtool-frontend service

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.

2 participants