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

fix notification when a survey is deleted, all collected data from the Survey is lost #4740 #810

Closed
wants to merge 2 commits into from

Conversation

Felysilver
Copy link
Contributor

@Felysilver Felysilver commented Mar 5, 2024

A confirmation modal comes up when deleting a Survey. And it has been displaying this to the UI "Deleting this survey will remove it from database. This action cannot be undone."

The requirement of this ushahidi/platform#4740 issue reported by @tuxpiper
was to change change this message to "Deleting this survey will remove it and all collected data from the database. This action cannot be undone." As I have implement. So with this everything must be fixed about this issue.

@Felysilver
Copy link
Contributor Author

@tuxpiper, @Angamanga, may you please look at my PR

@MugoBrian
Copy link
Contributor

Hello @Felysilver. The link redirects to this link https://github.com/ushahidi/platform-client-mzima/pull/url which results to " 404 page not found".

Could you confirm this?

@Angamanga
Copy link
Contributor

@Felysilver Thank you! I will review shortly!

@Felysilver
Copy link
Contributor Author

Thanks @MugoBrian I think the problem was the implementation of markdown and I decided to directly paste in the link. Now it is working.

@Angamanga
Copy link
Contributor

Thank you for the PR @Felysilver! Please add a short step by step checklist on how to test your changes when reviewing the PR 👍

@Angamanga Angamanga requested a review from ushahidlee March 16, 2024 08:15
@Angamanga
Copy link
Contributor

@ushahidlee Could you review this one please?

Copy link
Contributor

@ushahidlee ushahidlee left a comment

Choose a reason for hiding this comment

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

Hi @Felysilver,

This change is great but you have modified the Arabic i18n json. Could you please make this change on en.json instead?

@Felysilver
Copy link
Contributor Author

Hi @Felysilver,

This change is great but you have modified the Arabic i18n json. Could you please make this change on en.json instead?
@ushahidlee, thanks for the review, I have update the pr to reflect the changes you asked for.

@Angamanga Angamanga closed this Jun 26, 2024
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.

6 participants