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

Fix coordinators table #629

Merged
merged 1 commit into from
Nov 21, 2024
Merged

Fix coordinators table #629

merged 1 commit into from
Nov 21, 2024

Conversation

Calebgisa72
Copy link
Contributor

@Calebgisa72 Calebgisa72 commented Nov 17, 2024

PR Description

This PR fixes the coordinator table by removing organisation and role fields and put cohort and actions fields

Description of tasks that were expected to be completed

  • Retrieve the coordinators with the cohorts they are assigned to
  • Drop coordinator
  • Undrop coordinator
  • Change Coordinator Cohort

How has this been tested?

  • Clone this repository and navigate to the project directory
  • Ensure You have backend service running and set your .env file using .env.example.

git clone https://github.com/atlp-rwanda/atlp-pulse-fn.git cd atlp-pulse-fn git checkout fix-coordinators-table npm i npm run dev or npm run test

Number of Commits

The number of commits should not exceed 2 commits. In case they are more than that, please take your time and squash them.

1

#Screenshots (If appropriate)
Screenshot 2024-11-17 201306

Please check this Checklist before you submit your PR:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My code generate no warnings
  • My test coverage meet the set test coverage threshold
  • There are no vulnerabilities
  • There are no conflicts with the base branch

Copy link

vercel bot commented Nov 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
metron-devpulse ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 21, 2024 8:43am

@Calebgisa72 Calebgisa72 force-pushed the fix-coordinators-table branch from b7a4db2 to e8b50a5 Compare November 21, 2024 08:42
@Salim-54 Salim-54 merged commit 8f45bdb into develop Nov 21, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants