Skip to content

Conversation

@Ismaelmurekezi
Copy link
Contributor

@Ismaelmurekezi Ismaelmurekezi commented Sep 13, 2024

PR Description

This PR is for correcting issue related to single trainee detail page by adding missing functionality. Also following best practices to avoid some error which was rising during test on github , improving user experience

Desciption of task on what was needed to be done

  • Refactoring
  • Refining User Interface
  • Fixing some issues

How to test this PR

Locally
-First ,you have to switch to fix-single-detail-page branch
-Login as admin then under trainee-applicants page you choose view
Email:[email protected]
Password:password123

-My branch: fix-single-trainee-detail-page

Deploy
-For deployed one ,you click issue link, then under vercel bot
-Click Preview , Login after creating account , then the rest is same as those of local

https://atlp-devpulse-fn-git-fix-sin-0e7ca9-devpulses-projects-62c56cf1.vercel.app/

Track PR

issue link:
#125

Before
Before

After

After

After3

@vercel
Copy link

vercel bot commented Sep 13, 2024

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

Name Status Preview Comments Updated (UTC)
atlp-devpulse-fn ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 25, 2024 10:08pm

@gitguardian
Copy link

gitguardian bot commented Sep 13, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Copy link
Contributor

@mugishaj092 mugishaj092 left a comment

Choose a reason for hiding this comment

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

Also rember to add other action button (Approve and Reject)

Copy link
Contributor

@Ishimwe7 Ishimwe7 left a comment

Choose a reason for hiding this comment

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

Please consider feedbacks given and resolve conflicts

@qlty-cloud-legacy
Copy link

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

View more on Code Climate.

@mugishaj092 mugishaj092 added Ready For Merge and removed Ready For Review Extra attention is needed labels Sep 26, 2024
@Salim-54 Salim-54 merged commit e30941c into develop Sep 26, 2024
7 checks passed
Jadowacu1 added a commit that referenced this pull request Sep 30, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Jadowacu1 added a commit that referenced this pull request Oct 1, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Jadowacu1 added a commit that referenced this pull request Oct 2, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Jadowacu1 added a commit that referenced this pull request Oct 2, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Jadowacu1 added a commit that referenced this pull request Oct 2, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Jadowacu1 added a commit that referenced this pull request Oct 2, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Jadowacu1 added a commit that referenced this pull request Oct 2, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Jadowacu1 added a commit that referenced this pull request Oct 4, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
Ishimwe7 pushed a commit that referenced this pull request Oct 10, 2024
* feature: improve trainee details page

* handling missing application info, also adding download functionality

* fixing error related to download and refactoring

* Update TrainneeDetails.tsx

* handling issues related to deployment

* Fix number can't be shared (#130)

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

* #102 sidebar links review (#128)

* fix: remove placeholder property

* fix duplicate links

---------

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

* Ft minimize dashboard menu #110 (#140)

* fix: remove placeholder property

* ft minimize dashboard menu

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard by icon and categorize into section

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* fix minimize dashboard scrollbar

* Fix layout spacing between sidebar and main content in AdminLayout

* new

* Fix layout spacing between sidebar and main content in AdminLayout

* fix layout

---------

Co-authored-by: ceelogre <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>

* #118 fx: builtinSuperAdminCreateProgram (#126)

* fix: remove placeholder property

* The built-in superadmin account cannot create a program

---------

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

* feature: improve trainee details page

* handling missing application info, also adding download functionality

* Update TrainneeDetails.tsx

* adding way to send email and other adjustments

* refining and fixing some issues

* Update webpack.config.js

* customizing way of sending email

* refactoring code to fix issue related to code climate

* fixing issue for deployment

* fixing issues related to refactoring

---------

Co-authored-by: MUGISHA Emmanuel <[email protected]>
Co-authored-by: Mugisha <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: ceelogre <[email protected]>
Co-authored-by: ManziPatrick <[email protected]>
Co-authored-by: Prince-Kid <[email protected]>
Co-authored-by: Mucyo Prince <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Niyonshuti Jean De Dieu <[email protected]>
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.