Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix(DOMA-10698): fixed some cases of unitName parsing #5500

Closed

Conversation

dkoviazin
Copy link
Contributor

  1. When unitName contains some keywords for unitType and they are not separated with spaces - this keyword will not be cleared
    For example
    unitName "НП10" - before was cleared to warehouse 10, now it will be warehouse НП10
    unitName "НП 10" - will still be: warehouse 10
  2. Fixed bug when
    unitName "клад. 10" - was converted to "ад. 10"

@dkoviazin dkoviazin added 👶 small Easy to review changes up to 50 lines of code 🐞 BUG Issues with confirmed bugs labels Nov 16, 2024
Comment on lines -5 to +9
warehouse: ['помещение', 'подвал', 'помещ', 'пом', 'кл', 'кладовка', 'кладовая', 'нп'],
warehouse: ['помещение', 'подвал', 'помещ', 'пом', 'кл', 'кладовка', 'кладовая', 'нп', 'клад'],
flat: ['квартира', 'кв', 'комн'],
}

const HOUSE_IDENTIFIERS = 'д|уч|дом|участок|двлд'
const HOUSE_IDENTIFIERS = 'д|уч|дом|участок|двлд|домовладение'
Copy link
Member

Choose a reason for hiding this comment

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

Don't we think to make that configurable through environment?

@dkoviazin dkoviazin force-pushed the fix/condo/DOMA-10698/fixes_in_unit_name_parsing branch from 25077f3 to dc05336 Compare November 25, 2024 04:53
Copy link

sonarcloud bot commented Nov 25, 2024

@dkoviazin dkoviazin closed this Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BUG Issues with confirmed bugs 👶 small Easy to review changes up to 50 lines of code
Development

Successfully merging this pull request may close these issues.

2 participants