Skip to content

Conversation

@mutsinziisaac
Copy link
Contributor

@mutsinziisaac mutsinziisaac commented Dec 4, 2024

PR Description

This PR adds functionality to allow users to update their blogs and modifies the relevant files to support this feature.

Description of tasks that were expected to be completed

  1. Implement the ability for users to update their own blogs.
  2. Adde richtext as the content input for both create and update of the blog.
  3. Ensure proper validation and authorization checks to prevent unauthorized edits.
  4. Previously, there was a vague option for users to update their blogs, leading to challenges in maintaining or correcting their content
  5. blogs are hidden by default, so only admin can unhide them.

How has this been tested?

Clone the repository.
Checkout the branch ft-updateblog.
Set up the environment variables as indicated in .env.example.
Run npm install followed by npm run dev to start the app.
Test the update functionality on the blog on apollo studio.

Deployed Link:

https://atlp-devpulse-fn-git-ft-updateblogs-devpulses-projects-62c56cf1.vercel.app/

Track PR (Issue number & link):

#285

ScreenShoot

Screenshot 2024-12-04 at 17 57 30

@vercel
Copy link

vercel bot commented Dec 4, 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 2:02pm

@mutsinziisaac mutsinziisaac linked an issue Dec 4, 2024 that may be closed by this pull request
@ncutixavier
Copy link
Contributor

why do you have delete commits in this PR?

@qlty-cloud-legacy
Copy link

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

View more on Code Climate.

Copy link
Contributor

@Prince-Kid Prince-Kid left a comment

Choose a reason for hiding this comment

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

Great Job @mutsinziisaac

@ncutixavier
Copy link
Contributor

  • You included someones commits, and if it's not going to be resolved, your PR can't be merged
  • Only way to fix this, Make sure that your PR has only changes you made

@Mugisha146
Copy link
Contributor

  • You included someones commits, and if it's not going to be resolved, your PR can't be merged
  • Only way to fix this, Make sure that your PR has only changes you made

He did not include someones commit due to PC issues he was having. We met, used my PC to fix the requested changes, and committed them. Please review the code and test it. Everything is fine.

@ncutixavier ncutixavier merged commit e2c5387 into develop Dec 6, 2024
7 checks passed
@ncutixavier ncutixavier deleted the ft-updateblogs branch December 6, 2024 06:49
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

DREAM DREAM TEAM Ready For Review Extra attention is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

author update blog

8 participants