Skip to content

Feature solr merged#1961

Merged
hfekete merged 12 commits intobcgov:feature-solr-mergedfrom
hfekete:feature-solr-merged
Feb 18, 2026
Merged

Feature solr merged#1961
hfekete merged 12 commits intobcgov:feature-solr-mergedfrom
hfekete:feature-solr-merged

Conversation

@hfekete
Copy link
Collaborator

@hfekete hfekete commented Feb 18, 2026

Issue #, if available:

Description of changes:
verify in dev that this issue is not breaking synonyms in new solr

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).

mengdong19 and others added 12 commits January 26, 2026 12:43
…cgov#1944)

* fix - Add dataclass-based dynamic attribute initialization

* test - Add unit tests for `PaymentToken` and `ReceiptResponse` dataclasses and refactor initialization logic

* refactor - Replace `from_dict` methods with Pydantic-based dataclass initialization across `PaymentToken` and `ReceiptResponse` for consistent and simplified validation

* test - Add unit tests for `PaymentInvoice` dataclass and refactor to use Pydantic initialization

* refactor - Update `links` field in Pydantic model to use alias "_links" for consistency

* test/refactor - Add `invoice` field support in `ReceiptResponse` with tests and update `links` alias logic

* refactor/tests - Update DB connection logic for tests, add refund field, and improve type handling in tests

* test - Enhance `PaymentInvoice` tests to validate numeric types and use `pytest.approx` for floating-point assertions

* test/refactor - Replace hardcoded assertions in `test_payment_invoice` with dynamic checks using `data` dictionary

* test - Add `PaymentRefundInvoice` dataclass, update `refund_payment` logic, and add unit tests
perf: configure container concurrency to 20 and max scale to 60 for Namex services
* 32058-Update "create account" url for email templates.

* Update create account links.
Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239-Fix CI - bad-designation-notifier

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32438-Add John Andrews to Reports

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32438-Add John Andrews to Reports

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32438-Add John Andrews to Reports

Signed-off-by: Chen <Steven.Chen@gov.bc.ca>

32239 Fix Bad Designation Norifier CI and add John Andrews to Reports
32239-Fix CI - bad-designation-notifier
- Introduced `convert_solr_doc` for transforming Solr add documents.
- Added `_convert_nr_doc` and `_convert_corp_doc` for handling NR and CORP sources.
- Implemented `_extract_date` utility for ISO datetime parsing.
- Updated `poetry.lock` with dependency upgrades.
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
3.1% Duplication on New Code (required ≤ 3%)
B Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@hfekete hfekete merged commit 7027ae8 into bcgov:feature-solr-merged Feb 18, 2026
7 of 8 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.

4 participants

Comments