Skip to content

Conversation

@berniceu
Copy link
Contributor

@berniceu berniceu commented Dec 5, 2024

PR Description

This PR fixes loading and disabling buttons on the ticket pages and modals

Description of tasks that were expected to be completed

The submission buttons should show a loader when submitting and be disabled for a better user experience

How can this been tested?

  • Clone repo
  • Checkout branch fix-ticket-loading
  • Make sure that you set the environment variables referring to .env.example
  • Run npm install, then npm run dev to check if the app is running successfully
  • Navigate to tickets for both an admin or applicant and you will see the loading when you do any submission

Track PR (issue number & link)

N/A

@vercel
Copy link

vercel bot commented Dec 5, 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 Dec 5, 2024 11:07am

@qlty-cloud-legacy
Copy link

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

View more on Code Climate.

@Salim-54 Salim-54 merged commit 1c84e26 into develop Dec 6, 2024
7 checks passed
bahati10 pushed a commit that referenced this pull request Dec 6, 2024
Salim-54 pushed a commit that referenced this pull request Dec 9, 2024
* ft: admin assign a cohort to a trainee (#251)

* schedule technical assessment

* Implement Interview inviation

* Implement Interview inviation

* * Fix(#266): resolved conflicts

* Fix(#266): added comments and reactions to a blog

* Fix: Added modal (#287)

* schedule technical assessment

* Fix (#285):author will be able to update blogs (#294)

* Fix ticket loading (#295)

---------

Co-authored-by: Saddock Kabandana <[email protected]>
Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Philimuhire <[email protected]>
Co-authored-by: Ndahimana Bonheur <[email protected]>
Co-authored-by: mutsinziisaac <[email protected]>
Co-authored-by: Bernice Uwituze <[email protected]>
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.

7 participants