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

#170 An author will be able to update a blog #171

Merged
merged 2 commits into from
Dec 4, 2024
Merged

Conversation

mutsinziisaac
Copy link
Contributor

@mutsinziisaac mutsinziisaac commented Dec 2, 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. Update backend routes and controllers to handle blog updates securely and efficiently.
  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-bn.onrender.com/

Track PR (Issue number & link):

#170

ScreenShoot

N/A

@Jadowacu1 Jadowacu1 self-requested a review December 3, 2024 08:51
@ceelogre ceelogre temporarily deployed to ft-updateblog - atlp-devpulse-bn PR #171 December 3, 2024 14:37 — with Render Destroyed
@render render bot temporarily deployed to ft-updateblog - dev-pulse-bn PR #171 December 3, 2024 14:41 Destroyed
@ceelogre ceelogre temporarily deployed to ft-updateblog - atlp-devpulse-bn PR #171 December 3, 2024 14:41 — with Render Destroyed

This comment was marked as resolved.

@Jadowacu1 Jadowacu1 self-requested a review December 4, 2024 09:19
@ncutixavier ncutixavier merged commit 183d2a9 into develop Dec 4, 2024
1 check passed
@ncutixavier ncutixavier deleted the ft-updateblog branch December 4, 2024 13:30
bahati10 pushed a commit that referenced this pull request Dec 6, 2024
* fix (170): an author will be able to update a blog

* fix (170): allow all users to create and update blogs
Salim-54 pushed a commit that referenced this pull request Dec 9, 2024
* Application cycle process on applicant

* implement Technical Interview invitation

* implement Technical Interview invitation

* Implement Interview inviation

* Pipeline envs (#163)

* fix: remove the rm step

* test: run on PR

* test: test pusher cluster env

* fix: add notification envs

* test: add pusher key

* fix: add the pusher app key

* fix: log container start output

* fix: remove the e flag

* fix: revert changes

* fix: re-add the pusher app key

* fix: add pusher app key value

* fix: typo

* fix: test env file

* fix: syntax

* fix: remove the env from run cmd

* fix: revert changes

* fix: revert changes

* fix: wrap run cmd in an if statement

* Revert "test: test pusher cluster env"

This reverts commit 57b2bbe.

* test: create env file

* test: add error handling

* fix: run on push (#176)

* fix(apply-jobpost): incorporate feedback (#168)

- remove 'submitted' status
- add comments for 'rejected status'

* fix: don't exit on cmd failure (#177)

* fix: don't exit on cmd failure

* fix: re-add values

* fix: add new variables

* test: add container name

* fix: match ports

* fix: add devpulse email

* fix: port

* fix: port

* fix: remove container name

* fix: remove extraneous node env

* fix: add jwt key

* fix: run on push

* #170 An author will be able to update a blog (#171)

* fix (170): an author will be able to update a blog

* fix (170): allow all users to create and update blogs

* Fix to change status in application stage process (#165)

* Ft: Enhance Backend documentation (#164)

* fix: improve on notifications (#175)

* Application cycle process on applicant

* Application cycle process on applicant

* Fixing conflicts and removing unnecessary files

---------

Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Samuel Nishimwe <[email protected]>
Co-authored-by: Joslyn Manzi Karenzi <[email protected]>
Co-authored-by: mutsinziisaac <[email protected]>
Co-authored-by: Emmanuel MUGISHA <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: Christian Iradukunda <[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.

Author should be able to update a blog
10 participants