Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bugfix/20344 history tab irrelevant past rejection #1483

Conversation

flutistar
Copy link
Collaborator

Issue #: 20344

Description of changes:

Removed special characters from query string for History tab.

Before cleaning.

image

After cleaning

image

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

@EPortman
Copy link
Collaborator

EPortman commented Jul 2, 2024

/gcbrun

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://namex-dev--pr-1483-abb7b7hb.web.app

@EPortman
Copy link
Collaborator

EPortman commented Jul 2, 2024

I also did the same thing Roland did, I started off rejecting 'FRIDAY FUNDAY EVENTS LTD' - 0531273
image

And then I went to examine 'FRIDAY FUNDAY EVENTS LTD' - 0809933. It did not display anything in the history tab.
image

On a side note, is this the behavior we want for the history tab? It seems like these are not relevant (20+ duplicates).
image

Copy link
Collaborator

@ozamani9gh ozamani9gh left a comment

Choose a reason for hiding this comment

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

LGTM

@ozamani9gh ozamani9gh merged commit b6bfcfc into bcgov:main Jul 3, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants