Skip to content

docs(share_medical_records): fix PatientData field types to match code#62

Merged
arihantbansal merged 1 commit intoarcium-hq:mainfrom
giwaov:fix/medical-records-readme-types
Mar 8, 2026
Merged

docs(share_medical_records): fix PatientData field types to match code#62
arihantbansal merged 1 commit intoarcium-hq:mainfrom
giwaov:fix/medical-records-readme-types

Conversation

@giwaov
Copy link
Contributor

@giwaov giwaov commented Feb 25, 2026

Summary

  • Fixed gender field type: u8bool
  • Fixed weight field type: u64u16

Issue

The PatientData struct in the README showed incorrect types that don't match the actual implementation in share_medical_records/encrypted-ixs/src/lib.rs.

Verification

Confirmed against actual struct definition:

pub struct PatientData {
    pub patient_id: u64,
    pub age: u8,
    pub gender: bool,     // README had u8
    pub blood_type: u8,
    pub weight: u16,      // README had u64
    pub height: u16,
    pub allergies: [bool; 5],
}

Test plan

  • Verify README struct now matches actual source code

The README showed gender as u8 (actually bool) and weight as u64
(actually u16), which would mislead developers working with the struct.
@arihantbansal arihantbansal merged commit 2cc4bfb into arcium-hq:main Mar 8, 2026
10 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.

2 participants