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

Update Button to "Add Your Organization" on Organizations Page #1201

Closed
wants to merge 5 commits into from
Closed

Conversation

azizChebbi
Copy link

Changed successfully!

Copy link
Contributor

@bhaggya bhaggya left a comment

Choose a reason for hiding this comment

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

hi azizChebbi
I assume you are working on issue #1197 which says you need to update the Organization page button text. So you need to go to pages folder and then navigate to Contributors folder and in index.js there is in that you need to change the buttonText to "Add your Organization".

You have updated the text in wrong place

Also in comments can you mention Closes #1197 which is the issue number. So it will be easy to close the issue.

@codeclimate
Copy link

codeclimate bot commented Mar 30, 2022

Code Climate has analyzed commit 85944b4 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 80.8% (0.0% change).

View more on Code Climate.

@smsada
Copy link
Member

smsada commented Apr 1, 2022

Hi @azizChebbi, thank you for your interest in the project and we'd love to have you on board. Unfortunately, there are problems with your PR that we'd need to work through as a team. We have some processes where we can give you more coaching but you need to join our Organization to be properly added into all the right groups. If you join the org then we can re-open the PR!

This is how to join our organization:
https://github.com/civictechindex/CTI-website-frontend/wiki/Joining-the-Team

I am closing this PR unless it needs to be later reopened.

@smsada smsada closed this Apr 1, 2022
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.

None yet

4 participants