Skip to content

Feat/translation #348

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

Merged
merged 3 commits into from
May 24, 2021
Merged

Feat/translation #348

merged 3 commits into from
May 24, 2021

Conversation

macintoshpie
Copy link
Contributor

Any background context you want to provide?

BuildingSync v3 is being developed with breaking changes from v2

What does this PR do?

Creates the translation file for transforming breaking v3 changes from v2 bsync instance documents.
This includes translations for the breaking changes that were already merged into v3:

How should this be manually tested?

See validation_spec.rb. You can run locally and see the translated v2 file in spec/output

What are the relevant tickets?

#347

@macintoshpie macintoshpie added the Schema: General General update to BuildingSync label May 17, 2021
Copy link
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

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

Looks sweet! Only one question that needs resolved...

# and point schemaLocation to it instead
schema_doc = Nokogiri::XML(File.read('BuildingSync.xsd'))

GEOJSON_XSD_PATH = 'geojson.xsd'
Copy link
Member

Choose a reason for hiding this comment

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

Isn't this supposed to be GBXML_XSD_PATH and not GEOJSON?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

whoops, not sure how that happened, I'll get that fixed

Copy link
Member

Choose a reason for hiding this comment

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

terminology leaking from other projects :(

```
Then transform your file(s) with
```bash
xsltproc [-o <output>] <stylesheet> <file> [<file> ...]
Copy link
Member

Choose a reason for hiding this comment

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

👍

</xsl:template>

<!-- Map "Repair leaks / seal ducts" to "Repair leaks in ducts" -->
<xsl:template match="auc:MeasureName[. = 'Repair leaks / seal ducts']">
Copy link
Member

Choose a reason for hiding this comment

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

noice!

@macintoshpie
Copy link
Contributor Author

@aviveiros11 or @JieXiong9119 can one of you also take a look at this?

Copy link

@aviveiros11 aviveiros11 left a comment

Choose a reason for hiding this comment

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

Worked as expected for me 🚀

@JieXiong9119
Copy link
Contributor

I just tested it on pc. Working fine!

@macintoshpie macintoshpie requested a review from nllong May 24, 2021 18:14
Copy link
Member

@nllong nllong left a comment

Choose a reason for hiding this comment

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

looks shweet! Thanks for fixing the GeoJSON name.

@macintoshpie macintoshpie merged commit 13556df into develop-v3 May 24, 2021
@macintoshpie macintoshpie deleted the feat/translation branch May 24, 2021 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Non-breaking Change Schema: General General update to BuildingSync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants