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

0.4.0 examples patch #740

Merged
merged 7 commits into from
Jan 22, 2025
Merged

0.4.0 examples patch #740

merged 7 commits into from
Jan 22, 2025

Conversation

kathryn-ods
Copy link
Contributor

Overview

What does this pull request do?
cherry picking from main

fixes errors in the 0.4.0 branch examples and sets up plausible tracking

How can a reviewer test or examine your changes?
I have run these through the DRT tool and checked these are the same as main

Who is best placed to review it?

Closes #

Translations

  • New or edited strings appearing as a result of this PR will be picked up for translation
  • I've notified the translation coordinator of any new strings that will need
    translating. See the Handbook

Documentation & Release

kd-ods and others added 6 commits January 8, 2025 12:01
* removing uneeded fields from conditional

* removing uneeded fields from conditional

* removing uneeded fields from conditional

* adding in required fields
* removing min and max from shares where exact is present

swapping 2 letter iso country for 3 digit

swapping 2 letter iso country for 3 digit

fixing shares and componentRecords

fixing midindent

removing duplicate statement Id

fixing midindent

fixing shares

fixing component records

* examples fixes

* examples fixes

* removing broken link

* readding broken link

* Update listed-company-exempt-from-disclosure.json

entity didn't have hasPublicListing true

* fixing indentation

* Update multiple-indirect-ownership.json

making statementDate consistent

* Update multiple-tax-residencies.json

making dates consistent

---------

Co-authored-by: kd-ods <[email protected]>
@kathryn-ods kathryn-ods requested a review from kd-ods January 8, 2025 12:32
@kathryn-ods kathryn-ods changed the base branch from main to 0.4.0 January 14, 2025 11:49
Copy link
Collaborator

@kd-ods kd-ods left a comment

Choose a reason for hiding this comment

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

@kathryn-ods - let's not backport the change to statement.json to the 0.4.0 branch. It's a normative change to the schema and so would require a patch release.

Can you remove that from this branch?

@kathryn-ods
Copy link
Contributor Author

@kathryn-ods - let's not backport the change to statement.json to the 0.4.0 branch. It's a normative change to the schema and so would require a patch release.

Can you remove that from this branch?

Apologies - think that was bundled with another commit and I accidentally added it. Reverted now

Copy link
Collaborator

@kd-ods kd-ods left a comment

Choose a reason for hiding this comment

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

This looks good.

(And I've just checked: I don't think any of the updated JSON files are actually the ones embedded in the BODS docs page... so we don't even need to check the rebuild of the ReadTheDocs project.)

@kd-ods
Copy link
Collaborator

kd-ods commented Jan 17, 2025

On merging this, we can close #738

@kathryn-ods kathryn-ods merged commit acdc7fd into 0.4.0 Jan 22, 2025
2 of 4 checks passed
@kathryn-ods kathryn-ods deleted the 0.4.0_examples_patch branch January 22, 2025 13:53
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