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

Admin - Remove Support for Private Organizations #1327

Closed
palisadoes opened this issue Dec 30, 2023 · 14 comments
Closed

Admin - Remove Support for Private Organizations #1327

palisadoes opened this issue Dec 30, 2023 · 14 comments
Assignees

Comments

@palisadoes
Copy link
Contributor

palisadoes commented Dec 30, 2023

Is your feature request related to a problem? Please describe.
We need to reduce and simplify some of the Talawa features.

The concept of private organizations is unlikely given that most organizations using Talawa will be non-profits where transparency will be important.

Therefore not just must the organizations be always public. They must also always be visible.

Describe the solution you'd like

Remove all support for private organizations in:

  1. Sections of screens that contain references this feature
  2. Any screens completely dedicated to this feature
  3. All supporting code

Other:

  1. All tests must be valid and pass
  2. No other functionality must be affected.

Describe alternatives you've considered

  • N/A

Approach to be followed (optional)

  • See above

Additional context
Related issues include:

  1. API - Remove Support for Private Organizations talawa-api#1604
  2. Admin - Remove Support for Private Organizations #1327
  3. Mobile - Remove Support for Private Organizations talawa#2287

Potential internship candidates

@aialok
Copy link
Contributor

aialok commented Dec 31, 2023

I would love to work on this issue : )

@aialok
Copy link
Contributor

aialok commented Jan 2, 2024

ispublic

  • To be more clear about the given issue, I want to know in the above given picutres I need to remove isPublic functionality from the create Organisation modal ?

@palisadoes
Copy link
Contributor Author

Please read the issue thoroughly

@palisadoes
Copy link
Contributor Author

Please refer to this discussion before proceeding

@palisadoes
Copy link
Contributor Author

FYI - The issue blocking this has been completed

@palisadoes
Copy link
Contributor Author

Reopening because the introspection test in the previous PR failed.

@aialok please submit a follow up PR to fix this

@aialok
Copy link
Contributor

aialok commented Jan 14, 2024

Hey @palisadoes, I checked why the test failed and found the issue in the talawa-api repository, not talawa-admin. The problem is that the workflow is cloning the main branch of talawa-api, in which schema.graphql is not up to date as changes are done in develop branch, because of this isPublic is not updated with userRegistrationRequired in schema.graphql, causing the test to fail.

Here's the workflow file:

Workflow File

Here's the schema.graphql file:

Schema.graphql

And the failing GraphQL inspector:

Failing GraphQL Inspector

@palisadoes
Copy link
Contributor Author

@aialok How can we clone the develop branch instead?

@palisadoes
Copy link
Contributor Author

I'll open an issue to fix this. The action needs to checkout the develop branch after cloning

@aialok
Copy link
Contributor

aialok commented Jan 14, 2024

I'll open an issue to fix this. The action needs to checkout the develop branch after cloning

Yeah or we can use directly git clone -b branch_name --single-branch https://github.com/username/repository.git

@palisadoes
Copy link
Contributor Author

The branch name needs to match in both repos so that it doesn't break in main when we do merges from develop.

@aialok
Copy link
Contributor

aialok commented Jan 14, 2024

The branch name needs to match in both repos so that it doesn't break in main when we do merges from develop.

I need some time to research about how to render dynamically branch name.
I will fix this issue by today afternoon (Indian standard time).

@palisadoes
Copy link
Contributor Author

palisadoes commented Jan 14, 2024

The branch name needs to match in both repos so that it doesn't break in main when we do merges from develop.

I need some time to research about how to render dynamically branch name. I will fix this issue by today afternoon (Indian standard time).

  1. Ask to be assigned the issue GitHub Action Introspection Test Failure #1429. Issues cannot be assigned to people not participating in the comments section.
  2. We'll assign it to the first requester as per our policy

@aialok
Copy link
Contributor

aialok commented Jan 21, 2024

The branch name needs to match in both repos so that it doesn't break in main when we do merges from develop.

@palisadoes please close this issue as it is completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants