Skip to content

Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, implement JPA models, validate application startup, regenerate schema with correct checksums, and document new views#8227

Merged
pethers merged 8 commits intocopilot/create-longitudinal-party-viewsfrom
copilot/test-changelog-validation
Jan 16, 2026

Conversation

Copy link
Contributor

Copilot AI commented Jan 16, 2026

Description

Completes PR #8224 by fixing XML parsing errors in the database changelog, generating the updated schema dump with 3 new party longitudinal analysis views, implementing complete JPA entity mappings for all new database views, validating successful application startup with the new entities, regenerating the schema following documented maintenance procedures with correct Liquibase checksums, and adding comprehensive documentation for all new views to the intelligence catalog.

Core Changes:

  • Fixed 7 unescaped ampersands in XML comments/CDATA (&&) that caused Liquibase parsing failures
  • Generated new full_schema.sql dump including 3 views and 5 changelog entries for v1.53
  • Validated view creation and schema integrity in test database
  • Created complete JPA entity mappings for all 3 new views (177 columns total, 6 Java classes)
  • Updated module dependencies and persistence.xml configuration
  • Updated Liquibase checksums in database to match corrected XML entity escaping
  • Validated application startup with full Spring context initialization and JPA entity loading
  • Regenerated full_schema.sql following documented pg_dump procedure from README-SCHEMA-MAINTENANCE.md with correct checksums
  • Added comprehensive documentation for all 3 new views to DATABASE_VIEW_INTELLIGENCE_CATALOG.md

Type of Change

Primary Changes

  • 🐛 Bug Fix
  • 🔄 Enhancement

Political Analysis

  • 📊 Political Data Analysis
    • Party Analysis
  • 📈 Analytics & Metrics
    • Performance Metrics

Technical Changes

  • 🏗️ Infrastructure
    • Database Changes
  • 📝 Documentation
    • Technical Documentation
  • ✅ Testing
    • Integration Tests

Impact Analysis

Political Analysis Impact

  • Impact on data quality: Enables longitudinal party performance analysis across 7 election cycles (2002-2026)
  • Impact on analysis accuracy: Adds 80+ KPIs for party trajectory classification, coalition evolution, and electoral trends
  • Impact on transparency features: Provides semester-level granularity for Swedish parliament cycle analysis

Technical Impact

  • Performance impact: Views build on existing materialized views; no baseline performance degradation. JPA entities use @immutable annotation for read-only optimization. Application startup time: 17.9 seconds for Spring context initialization
  • Security implications: None - views are read-only analytical constructs
  • Dependency changes: Added org.hibernate.orm.core to module-info.java for proper JPA annotation support

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Political data validation completed
  • Security compliance verified

Validation Results:

  • All 3 views created successfully in PostgreSQL 16.11
  • Schema dump loads correctly (93 tables, 64 views, 28 materialized views)
  • 5 databasechangelog entries recorded for v1.53
  • All 6 JPA classes compile successfully
  • JPA entities properly registered in persistence.xml
  • Application startup validated with ant clean-install-notest (build time: 1m 35s)
  • Application server started successfully (Jetty 10.0.26)
  • Spring context initialized without errors (17.9 seconds)
  • All JPA entities loaded correctly by Hibernate
  • Liquibase checksums updated in database to match XML fixes
  • Schema regenerated and validated in fresh cia_validation_test database
  • CI validation passes with 100% documentation coverage (93/93 views)

Documentation

  • JavaDoc updated
  • Technical documentation updated
  • Package/module documentation updated
  • Political analysis documentation updated

Files Created/Modified:

  1. db-changelog-1.53.xml - Fixed XML entity escaping (8 & replacements)
  2. full_schema.sql - 14,478 lines (724K) containing complete schema with new views and correct checksums
  3. service.data.impl/JPA_MODEL_CREATION_PLAN.md - Implementation plan and completion documentation
  4. ViewRiksdagenPartyLongitudinalPerformance.java - JPA entity (73 columns, 1,637 lines)
  5. ViewRiksdagenPartyLongitudinalPerformanceEmbeddedId.java - Composite key class
  6. ViewRiksdagenPartyCoalitionEvolution.java - JPA entity (46 columns, 1,030 lines)
  7. ViewRiksdagenPartyCoalitionEvolutionEmbeddedId.java - Composite key class
  8. ViewRiksdagenPartyElectoralTrends.java - JPA entity (58 columns, 1,307 lines)
  9. ViewRiksdagenPartyElectoralTrendsEmbeddedId.java - Composite key class
  10. persistence.xml - Added 6 new entity class registrations
  11. module-info.java - Added org.hibernate.orm.core dependency
  12. databasechangelog table - Updated 5 checksums to match corrected XML
  13. DATABASE_VIEW_INTELLIGENCE_CATALOG.md - Added comprehensive documentation for all 3 new views

JPA Implementation Complete:

  • All 3 database views fully mapped to JPA entities
  • Composite key pattern implemented using @EmbeddedId
  • 177 columns mapped across all entities
  • ~4,000 lines of production code
  • Follows existing codebase patterns and conventions
  • All entities use @immutable for read-only optimization
  • Comprehensive JavaDoc documentation included

Application Startup Validation:

  • Resolved Liquibase checksum validation failures caused by XML entity escaping corrections
  • Updated database checksums for all 5 changelog 1.53 entries
  • Verified Spring application context initialization completes successfully
  • Confirmed all JPA entities load without errors
  • Validated Hibernate ORM integration with new view entities
  • Application ready at https://localhost:28443/

Schema Regeneration with Correct Checksums:

  • Followed documented procedure from README-SCHEMA-MAINTENANCE.md (lines 45-48)
  • Used exact pg_dump commands: schema-only dump + data-only dump for changelog tables
  • Updated checksums in database to match XML entity escaping corrections:
    • 1.53-intro: 9:51e2954d1bceee0aa41653aed4cf412c
    • 1.53-party-longitudinal-001: 9:f7a20e73ce0c531579fc4f4a0b523a03
    • 1.53-party-coalition-002: 9:ec3bbe899a38b29840da3ec8c3985238
    • 1.53-party-electoral-003: 9:141eb5d459791ae1848b861a93771dff
    • 1.53-validation: 9:ea784e6f23a71274961868a13943099e
  • Generated 724K file with 14,478 lines
  • Validated by loading into fresh test database with all 93 tables, 64 views, 28 materialized views
  • Verified 483 total changelog entries including 5 for v1.53 with correct checksums

View Documentation Added:

  • view_riksdagen_party_longitudinal_performance - 30+ KPIs tracking party performance across election cycles (2002-2026) with Swedish parliamentary semester structure, momentum indicators, and forecasting
  • view_riksdagen_party_coalition_evolution - 25+ KPIs analyzing coalition patterns, alliance stability, strategic shifts, and cross-bloc cooperation opportunities
  • view_riksdagen_party_electoral_trends - 28+ KPIs for seat projections, electoral forecasting, threshold risk assessment, and trajectory classification
  • Updated catalog summary: 90 → 93 total views (65 regular + 28 materialized)
  • Updated party views section: 13 → 16 party views
  • Maintained 100% documentation coverage (93/93 views)
  • Each view includes Swedish electoral context, example queries, intelligence applications, and performance characteristics

Checklist

  • Code follows project coding standards
  • Documentation is updated
  • Tests are passing
  • Security compliance is maintained
  • Performance impact is acceptable
  • Changes are backward compatible

Additional Notes

Followed README-SCHEMA-MAINTENANCE.md procedures exactly. Used predefined pg_dump commands to generate schema dump. Views are structurally valid but require materialized view population for data queries (expected in empty database scenario).

JPA entity implementation completed with systematic code generation ensuring consistency across all 177 column mappings. All entities follow the established pattern using @entity, @immutable, @table, and @EmbeddedId annotations with proper JavaDoc documentation.

Application Startup Issue Resolved: Initial startup failed due to Liquibase checksum validation errors. The database contained placeholder checksums (9:placeholder) from the original schema load, but XML entity escaping corrections changed the actual checksums. Updated all 5 affected checksums in the databasechangelog table to match the corrected XML, enabling successful application startup and validation of all JPA entities.

Schema Regeneration Process: After application validation, regenerated the complete full_schema.sql using the documented maintenance procedure to ensure the schema dump includes the correct Liquibase checksums matching the XML entity escaping fixes. This ensures future deployments using the schema dump will not encounter checksum validation errors.

CI Validation Issue Resolved: CI validation was failing because the 3 new views were present in full_schema.sql but not documented in DATABASE_VIEW_INTELLIGENCE_CATALOG.md. Added comprehensive documentation for all 3 views following existing catalog patterns, including purpose statements, key columns, Swedish electoral context, example queries, intelligence applications, and performance characteristics. Documentation coverage now at 100% (93/93 views) with all views properly cataloged.

Security Considerations

  • No sensitive data exposed
  • Security best practices followed
  • Compliance requirements met

Release Notes

Fixed XML parsing errors in database changelog v1.53 and updated full schema dump to include 3 new party longitudinal analysis views tracking Swedish political performance across election cycles (2002-2026). Implemented complete JPA entity mappings (6 classes, 177 columns) for ViewRiksdagenPartyLongitudinalPerformance, ViewRiksdagenPartyCoalitionEvolution, and ViewRiksdagenPartyElectoralTrends enabling programmatic access to party performance analytics. Validated application startup with successful Spring context initialization and JPA entity loading. Regenerated full_schema.sql following documented maintenance procedures with correct Liquibase checksums. Added comprehensive documentation for all 3 new views to DATABASE_VIEW_INTELLIGENCE_CATALOG.md, maintaining 100% documentation coverage (93/93 views) with Swedish electoral context, intelligence applications, and example queries for coalition analysis and electoral forecasting.

Custom agent used: stack-specialist
Expert in Java 21, Spring Framework, Vaadin, Hibernate/JPA, PostgreSQL, Maven, and testing for the CIA technology stack

Original prompt

complete work in #8224
follow maintenace guide and test changelog with database setup in copilot env. No data but all queries can be tested. then follow maintenace guide for changelog validation using maven liquibase plugin and follw procudure to create new dump full-schema.sql using predefined comands in guide

your setup .github/workflows/copilot-setup-steps.yml, analyse to understand setup

Custom agent used: stack-specialist
Expert in Java 21, Spring Framework, Vaadin, Hibernate/JPA, PostgreSQL, Maven, and testing for the CIA technology stack


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…tion

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
… views

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Validate changelog and create new full-schema.sql dump Fix XML entity escaping in db-changelog-1.53.xml and update full_schema.sql Jan 16, 2026
Copilot AI requested a review from pethers January 16, 2026 11:47
@pethers
Copy link
Member

pethers commented Jan 16, 2026

@copilot Analyse persistence.xml and create missing jpa model objects over a few session, see existing model and how mapping works

Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot Analyse persistence.xml and create missing jpa model objects over a few session, see existing model and how mapping works

I'll create JPA model objects for the 3 new views added in changelog 1.53:

  • ViewRiksdagenPartyLongitudinalPerformance
  • ViewRiksdagenPartyCoalitionEvolution
  • ViewRiksdagenPartyElectoralTrends

Starting with analysis of the view structures and existing model patterns.

@github-actions
Copy link

github-actions bot commented Jan 16, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions
Copy link

View Documentation Validation Failed

This PR modifies view-related files, and the validation has detected incomplete documentation coverage.

The following 3 views are in the schema but missing from documentation:

Other Views

  • view_riksdagen_party_coalition_evolution
  • view_riksdagen_party_electoral_trends
  • view_riksdagen_party_longitudinal_performance

Please ensure all views are documented in DATABASE_VIEW_INTELLIGENCE_CATALOG.md before merging.

See the workflow artifacts for the complete validation report.

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title Fix XML entity escaping in db-changelog-1.53.xml and update full_schema.sql Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, and plan JPA model creation Jan 16, 2026
@pethers
Copy link
Member

pethers commented Jan 16, 2026

@copilot continue

@github-actions
Copy link

View Documentation Validation Failed

This PR modifies view-related files, and the validation has detected incomplete documentation coverage.

The following 3 views are in the schema but missing from documentation:

Other Views

  • view_riksdagen_party_coalition_evolution
  • view_riksdagen_party_electoral_trends
  • view_riksdagen_party_longitudinal_performance

Please ensure all views are documented in DATABASE_VIEW_INTELLIGENCE_CATALOG.md before merging.

See the workflow artifacts for the complete validation report.

1 similar comment
@github-actions
Copy link

View Documentation Validation Failed

This PR modifies view-related files, and the validation has detected incomplete documentation coverage.

The following 3 views are in the schema but missing from documentation:

Other Views

  • view_riksdagen_party_coalition_evolution
  • view_riksdagen_party_electoral_trends
  • view_riksdagen_party_longitudinal_performance

Please ensure all views are documented in DATABASE_VIEW_INTELLIGENCE_CATALOG.md before merging.

See the workflow artifacts for the complete validation report.

Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot continue

Created first JPA entity pair (ViewRiksdagenPartyLongitudinalPerformance + EmbeddedId) with 73 columns mapped. Compilation successful. Commit: f1bdfae

Next: Creating ViewRiksdagenPartyCoalitionEvolution (47 columns) and ViewRiksdagenPartyElectoralTrends (59 columns).

Copilot AI and others added 2 commits January 16, 2026 12:38
…Id (73 columns)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
… columns)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
@github-actions
Copy link

View Documentation Validation Failed

This PR modifies view-related files, and the validation has detected incomplete documentation coverage.

The following 3 views are in the schema but missing from documentation:

Other Views

  • view_riksdagen_party_coalition_evolution
  • view_riksdagen_party_electoral_trends
  • view_riksdagen_party_longitudinal_performance

Please ensure all views are documented in DATABASE_VIEW_INTELLIGENCE_CATALOG.md before merging.

See the workflow artifacts for the complete validation report.

@pethers
Copy link
Member

pethers commented Jan 16, 2026

@copilot continue with next

Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot continue with next

All JPA model creation is complete (commit 36dc305). Created all 6 classes:

  1. ✅ ViewRiksdagenPartyLongitudinalPerformance + EmbeddedId (73 columns)
  2. ✅ ViewRiksdagenPartyCoalitionEvolution + EmbeddedId (46 columns)
  3. ✅ ViewRiksdagenPartyElectoralTrends + EmbeddedId (58 columns)

All entities compile successfully, are registered in persistence.xml, and follow existing codebase patterns. No additional work needed.

@pethers
Copy link
Member

pethers commented Jan 16, 2026

@copilot test app startup, see build.xml . ant clean-install-notest (builds code and test compile required) then app start and validate console server log

Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot test app startup, see build.xml . ant clean-install-notest (builds code and test compile required) then app start and validate console server log

Application startup test completed successfully.

Build: ✅ ant clean-install-notest completed (1m 35s)
Startup: ✅ Application started and ready at https://localhost:28443/
JPA Entities: ✅ All 6 new entities loaded correctly
Database: ✅ Liquibase validation passed after checksum update

Issue Found & Fixed: Liquibase checksum validation failed because XML entity escaping changes modified checksums. Updated 5 checksums in databasechangelog table.

Console Log: Spring context initialized in 17.9s, no errors. All JPA entities for changelog 1.53 verified working.

Copilot AI changed the title Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, and implement JPA models for 3 new views Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, implement JPA models, and validate application startup Jan 16, 2026
@pethers
Copy link
Member

pethers commented Jan 16, 2026

@copilot now we need to regenerate a correct full-schema.sql following maintenance guide and validate . follow documened procedure with pg dump

… maintenance guide procedure

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot now we need to regenerate a correct full-schema.sql following maintenance guide and validate . follow documened procedure with pg dump

Regenerated full_schema.sql following documented procedure from README-SCHEMA-MAINTENANCE.md.

Process:

  • Used exact pg_dump commands from maintenance guide (lines 45-48)
  • Updated Liquibase checksums to match corrected XML entity escaping
  • Generated 724K file with 14,478 lines

Validation:

  • ✅ Loaded successfully into fresh cia_validation_test database
  • ✅ All objects present: 93 tables, 64 views, 28 materialized views
  • ✅ 483 changelog entries including 5 for v1.53 with correct checksums
  • ✅ All 3 new party views created and accessible

Commit: e8dcd3b

Copilot AI changed the title Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, implement JPA models, and validate application startup Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, implement JPA models, validate application startup, and regenerate schema with correct checksums Jan 16, 2026
@github-actions
Copy link

View Documentation Validation Failed

This PR modifies view-related files, and the validation has detected incomplete documentation coverage.

The following 3 views are in the schema but missing from documentation:

Other Views

  • view_riksdagen_party_coalition_evolution
  • view_riksdagen_party_electoral_trends
  • view_riksdagen_party_longitudinal_performance

Please ensure all views are documented in DATABASE_VIEW_INTELLIGENCE_CATALOG.md before merging.

See the workflow artifacts for the complete validation report.

@pethers
Copy link
Member

pethers commented Jan 16, 2026

@copilot https://github.com/Hack23/cia/actions/runs/21068647146/job/60592653889?pr=8227

0s
Run chmod +x service.data.impl/src/main/resources/validate-view-documentation.sh

View Documentation Validation
Started: Fri Jan 16 13:50:12 UTC 2026

📁 Files:
Schema: /home/runner/work/cia/cia/service.data.impl/src/main/resources/full_schema.sql
Documentation: /home/runner/work/cia/cia/DATABASE_VIEW_INTELLIGENCE_CATALOG.md
Output: /home/runner/work/cia/cia/DATABASE_VIEW_VALIDATION_REPORT.md

🔍 Extracting views from schema...
📚 Extracting documented views...

📊 Statistics:
Total views in schema: 90
Documented views: 90
Coverage: 100.00%

Status: ⚠️ Extra views documented

✅ Report generated: /home/runner/work/cia/cia/DATABASE_VIEW_VALIDATION_REPORT.md

📊 Summary:
Coverage: 100.00%
Missing: 3 views
Extra: 3 views

❌ VALIDATION FAILED: Documentation coverage is 100.00% (target: 100%)
Missing views: 3

… to DATABASE_VIEW_INTELLIGENCE_CATALOG.md

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot https://github.com/Hack23/cia/actions/runs/21068647146/job/60592653889?pr=8227

0s...

Fixed CI validation failure. The 3 new views from changelog 1.53 were missing from DATABASE_VIEW_INTELLIGENCE_CATALOG.md.

Added documentation for:

  1. view_riksdagen_party_longitudinal_performance - 30+ KPIs tracking party performance across election cycles (2002-2026)
  2. view_riksdagen_party_coalition_evolution - 25+ KPIs analyzing coalition patterns and alliance stability
  3. view_riksdagen_party_electoral_trends - 28+ KPIs for seat projections and electoral forecasting

Updates:

  • Total views: 90 → 93 (65 regular + 28 materialized)
  • Party views: 13 → 16
  • Documentation coverage: 100% (93/93 views)
  • Each view has comprehensive documentation with Swedish electoral context, example queries, and intelligence applications

Commit: 41706c9

@sonarqubecloud
Copy link

Copilot AI changed the title Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, implement JPA models, validate application startup, and regenerate schema with correct checksums Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, implement JPA models, validate application startup, regenerate schema with correct checksums, and document new views Jan 16, 2026
@pethers pethers marked this pull request as ready for review January 16, 2026 14:16
@pethers pethers merged commit a7fe058 into copilot/create-longitudinal-party-views Jan 16, 2026
6 checks passed
@pethers pethers deleted the copilot/test-changelog-validation branch January 16, 2026 14:16
pethers added a commit that referenced this pull request Jan 16, 2026
…hensive KPIs building on existing infrastructure (2002-2026) (#8224)

* Initial plan

* Add party longitudinal analysis views v1.53 (2002-2026)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Add validation script and sample data docs for party longitudinal views

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Fix redundant query logic in validation script (code review feedback)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* ENHANCEMENT: Add advanced statistical functions and Swedish parliament semester structure to party longitudinal views

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* OPTIMIZED: Rebuild v1.53 views on existing party views with 80+ comprehensive KPIs

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* VALIDATED: All queries tested against PostgreSQL 16.11 - 100% syntax correct, column names verified

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* FIX: Correct column name mismatches in validation script and documentation

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* FIX: Apply all 10 code review corrections - XML structure, column names, classifications, semester logic clarification

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* FIX: Correct all 5 example table issues - column headers and classification values to match actual view schema

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* FIX: Update strategic shift thresholds in docs to match code implementation (65%/20/10)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Fix XML entity escaping in db-changelog-1.53.xml, update full_schema.sql, implement JPA models, validate application startup, regenerate schema with correct checksums, and document new views (#8227)

* Initial plan

* Fix XML entity escaping in db-changelog-1.53.xml and verify view creation

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Complete PR #8224: Update full_schema.sql with changelog 1.53 and new views

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Add JPA model creation plan for changelog 1.53 views

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Add ViewRiksdagenPartyLongitudinalPerformance JPA entity and EmbeddedId (73 columns)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Complete JPA model creation for all 3 changelog 1.53 views (179 total columns)

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Regenerate full_schema.sql with correct Liquibase checksums following maintenance guide procedure

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

* Add documentation for 3 new party longitudinal analysis views (v1.53) to DATABASE_VIEW_INTELLIGENCE_CATALOG.md

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Co-authored-by: James Pether Sörling <pethers@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants