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

filter unstructured posts #4663

Merged
merged 3 commits into from
Nov 13, 2023
Merged

filter unstructured posts #4663

merged 3 commits into from
Nov 13, 2023

Conversation

Mh-Asmi
Copy link
Contributor

@Mh-Asmi Mh-Asmi commented Nov 3, 2023

This pull request makes the following changes:

Test checklist:

  • [ ]

  • I certify that I ran my checklist

Fixes ushahidi/platform# .

Ping @ushahidi/platform

Copy link

linear bot commented Nov 3, 2023

USH-947 No one is able to see unstructured posts

How to reproduce

In a deployment that has Africa's Talking enabled (i.e. mzima-release).

  1. From your command line, send a webhook request (mimicking Africa's Talking service) to have an unstructured post created as it came from SMS.

    curl -X 'POST' 'https://mzima-release-api.staging.ush.zone/sms/africas-talking' \
       -H 'connection: close' \
       -H 'content-type: application/x-www-form-urlencoded' -H 'user-agent: at-hydra-api/1.0' \
       -d $'linkId=f7a20132-41a4-462e-90c7-f6186b980211&text=Test+message+'$RANDOM'&to=22686&id=dc833ff8-eaf7-4f0d-9d79-22bdb2f27672&date=2023-10-26+08%3A59%3A38&from=%2B25401010101'
    
  2. As an admin user in the deployment, see the list of posts, your test message should appear. If it doesn't, you have just reproduced the bug.

How it's not a problem with the SMS provider

  1. In Africa's Talking data source configuration, enable 'Import to Survey'
  2. Send the webhook request
  3. Your post will most certainly appear in the list of posts now.

@Mh-Asmi Mh-Asmi requested a review from webong November 3, 2023 04:36
Copy link
Collaborator

@webong webong left a comment

Choose a reason for hiding this comment

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

This is good to go now!

@webong webong merged commit ae24ae8 into develop Nov 13, 2023
4 checks passed
Mh-Asmi added a commit that referenced this pull request Nov 20, 2023
* fix create category by comment user-id

* add include_unstructured_posts filter for post

* chore: update fetch survey to add unknown form and revert some previous changes

---------

Co-authored-by: Wisdom Ebong <wisdom@ushahidi.com>
tuxpiper pushed a commit that referenced this pull request Nov 20, 2023
* fix create category by comment user-id

* add include_unstructured_posts filter for post

* chore: update fetch survey to add unknown form and revert some previous changes

---------

Co-authored-by: Wisdom Ebong <wisdom@ushahidi.com>
@Mh-Asmi Mh-Asmi deleted the USH-947 branch May 28, 2024 06:36
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.

None yet

2 participants