Skip to content

Additional Admin Search Functionality#44

Merged
melkuo merged 5 commits intoBuildCanada:mainfrom
nwarwick:feature/search
Jul 7, 2025
Merged

Additional Admin Search Functionality#44
melkuo merged 5 commits intoBuildCanada:mainfrom
nwarwick:feature/search

Conversation

@nwarwick
Copy link
Copy Markdown
Contributor

@nwarwick nwarwick commented Jul 1, 2025

Re #33

  • Adds search functionality to Promises, Feeds, and Ministers
  • Fixes a typo in the compound_name method for Ministers
  • Linting fixes

compound_name before:
Screenshot 2025-07-01 at 10 10 30 AM

compound_name after:
Screenshot 2025-07-01 at 10 10 17 AM

@nwarwick nwarwick force-pushed the feature/search branch 4 times, most recently from c51dca4 to 5b7e6f5 Compare July 1, 2025 14:20
@nwarwick
Copy link
Copy Markdown
Contributor Author

nwarwick commented Jul 1, 2025

If #45 is merged before, I can add a test for the now fixed compound_name method.

@melkuo melkuo merged commit 51d9f83 into BuildCanada:main Jul 7, 2025
3 checks passed
@melkuo
Copy link
Copy Markdown
Contributor

melkuo commented Jul 7, 2025

Missed this one, thanks!

Entries and Evidences up next if you're still up for it @nwarwick ?

@nwarwick
Copy link
Copy Markdown
Contributor Author

nwarwick commented Jul 7, 2025

On it @melkuo!

@nwarwick
Copy link
Copy Markdown
Contributor Author

nwarwick commented Jul 7, 2025

PR up @melkuo: #50

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