Skip to content

Conversation

@seaerchin
Copy link
Collaborator

Problem

https://opengovproducts.slack.com/archives/C0949RH4NTA/p1752216333458559

Solution

  • don't use helpers.message because it's compute intensive
  • return a hardcoded message cos it won't be seen by frontend
  • update initial regex for creation of url to follow frontend's requirements

@adriangohjw
Copy link
Contributor

bugbot run

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ BugBot reviewed your changes and found no bugs!


BugBot free trial expires on July 22, 2025
You have used $0.00 of your $50.00 spend limit so far. Manage your spend limit in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎

Copy link
Collaborator

@harishv7 harishv7 left a comment

Choose a reason for hiding this comment

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

lgtm, pls test on staging first

@seaerchin seaerchin merged commit 80942e7 into develop Jul 15, 2025
38 of 41 checks passed
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