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

Create in-ita.json #533

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Create in-ita.json #533

wants to merge 1 commit into from

Conversation

IsabelBirds
Copy link
Collaborator

Made a start as discussed - I wasn't able to find much more information.

@IsabelBirds IsabelBirds requested a review from kd-ods April 13, 2023 15:18
@IsabelBirds IsabelBirds self-assigned this Apr 13, 2023
Copy link
Contributor

@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.

In addition to the couple of specific field-level comments, there's one other thing. Can you edit the filename to 'in-ita.json' please?

You may also want to change the code and filename to in-trn if you agree with my naming suggestion.

Also - do you have any examples of Trust registration numbers at all? I have questions about whether they really are issued at a national level or whether there is State/Provincial involvement. For example, I found: https://charity.maharashtra.gov.in:8060/knowyourtrust.aspx. But maybe a Charitable Trust is different from the types of trust set up under the ITA.

I think we need to have some certainty that there is national oversight and management of these Trust registration numbers before publishing this list. What do you think @IsabelBirds?

@@ -0,0 +1,48 @@
{
"name": {
"en": "Indian Trust Act ",
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe call this 'Trust Registration Number, India'?

],
"sector": [],
"code": "IN-ITA",
"confirmed": false,
Copy link
Contributor

Choose a reason for hiding this comment

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

You can set "confirmed" to true. (The PR review process does the job of blocking the list from publication until approved. )

@kd-ods
Copy link
Contributor

kd-ods commented Aug 23, 2023

Hi @IsabelBirds - I just saw that this is still open. Do you want to pick it up again?

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