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: translate error message #310

Merged
merged 1 commit into from
Oct 4, 2023
Merged

fix: translate error message #310

merged 1 commit into from
Oct 4, 2023

Conversation

rwamugema
Copy link
Contributor

@rwamugema rwamugema commented Oct 3, 2023

PR Description

Add proper error message when user have not added to any program or cohort yet, and translate it.

Description of tasks that were expected to be completed

  1. Add proper error message
  2. Translate it.

How has this been tested?

Login to any organization with any email which have not added to any program or cohort.

Number of Commits

1

Screenshots (If appropriate)

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

@vercel
Copy link

vercel bot commented Oct 3, 2023

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 Oct 3, 2023 0:20am

Copy link

@Virgile1k Virgile1k left a comment

Choose a reason for hiding this comment

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

Checked and approved

@ceelogre ceelogre linked an issue Oct 4, 2023 that may be closed by this pull request
@ceelogre ceelogre merged commit 308ee11 into develop Oct 4, 2023
4 checks passed
@ceelogre ceelogre deleted the fix-popup-message branch October 4, 2023 13:23
NTElissa pushed a commit that referenced this pull request Oct 23, 2023
NTElissa pushed a commit that referenced this pull request Oct 23, 2023
NTElissa pushed a commit that referenced this pull request Oct 23, 2023
Ft docs implementation (#254)

* fix login

* docs implementation

github activity (#244)

ft:Upload Profile

Co-authored-by: teerenzo <[email protected]>

Dark theme implementation (#259)

* ch(fix sign up):  fixing sign up

- google is removed
-Date of birth is validated to be at least 18 years
-Email field is automatically filled and uneditable ( collect the email
from the sign up link sent to them)
-Password should not be too easy. (Should not be in the first 100 most
common passwords: https://tinyurl.com/wiki-100)

[Finishes #42-signup-fix]

* ch(dark-theme):implementation of dark theme

-Dark theme is added using our new design
-implement effective dark theme at all

[finishes #59-dark-theme-implementation]

* add orgLogin update (#260)

* add

* Add change

* move space in organiaztionLogin

* add newUpdate

* add newChange

* add change

* add changing

* Add change

* add change

---------

Co-authored-by: Rachel UMWARI <[email protected]>

* style: enhance UI/UX on the super admin dashboard

- Organization page table design

[Fixes #65]

* style: enhance UI/UX on the super admin dashboard

- Changed Organization page table design to use the DataTable component.
- Fixed some styling and formatting errors
- Fixed grammatical errors in code.

[Fixes #65]

fix: translate error message (#310)

fix-trainee-dashbord (#312)

-modifying trainee dashbord designs

fix: Replace buttons with icons on organizations page (#318)

- Change action buttons to specific icons
- Change icon colors to our platform color codes

add retrieving coordinators from data (#317)

Feat(drop trainee): Admin/coordinator to drop trainee (#281)

* Rating Feature Implementation

* feat: drop trainee

---------

Co-authored-by: Aline096 <[email protected]>

fix:new cohort visibility (#326)

Co-authored-by: Your Name <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unhelpful message
4 participants