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

bg pdf/csv export button #64

Closed
wants to merge 1 commit into from
Closed

Conversation

masterchief-00
Copy link
Contributor

@masterchief-00 masterchief-00 commented Aug 7, 2023

What does this PR do?

  • adds the ability to export trainee details to PDF and CSV file

Description of Task to be completed?

  • N/A

How should this be manually tested?

  • navigate to the trainee-applicants table
  • locate the actions column
  • click the three dots and choose view
  • on the single applicants view page, scroll to the bottom
  • 2nd button from the left, choose the export option(csv or pdf)
  • there you go :)

Any background context you want to provide?

  • N/A

What are the relevant pivotal tracker stories?
N/A

Screenshots (if appropriate)

Screenshot 2023-08-07 160246

Questions:

  • N/A

@vercel
Copy link

vercel bot commented Aug 7, 2023

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

Name Status Preview Comments Updated (UTC)
atlp-devpulse-fn ❌ Failed (Inspect) Oct 26, 2023 8:22pm

@masterchief-00 masterchief-00 added the Ready For Review Extra attention is needed label Sep 4, 2023
@Anguandia
Copy link
Member

@masterchief-00 please look into this PR and advise on the way forward with it

@codeclimate
Copy link

codeclimate bot commented Oct 26, 2023

Code Climate has analyzed commit 5a26ec7 and detected 5 issues on this pull request.

Here's the issue category breakdown:

Category Count
Complexity 2
Duplication 2
Clarity 1

View more on Code Climate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants