Skip to content

[front] Twitter sync message Strapi #5506

Ben requested to merge feat/strapi-twitter-sync-5506 into master

Ticket(s) / Related Merge Requests

Summary of Changes

Twitter sync message Strapi #5506 (closed)

Also fixes broken Twitter Sync on production.

Testing Considerations

Go to connect your Twitter to sync and verify the URL comes from Strapi. I prefixed the Strapi driven text so you know that is the source and not the default message - it reads Strapi verifying my channel on @minds. {url}.

Test verification actually works too as this also fixes an issue where-in that is not working in prod because of changes to the Twitter API

Note the text is CMS driven and I've set the FALLBACK text if nothing is supplied by Strapi to be the text in the issue card. To illustrate Strapi is working, I've changed the CMS driven text, which can be changed to anything very easily.

Deployment Considerations

Standard deployment - need to deploy Strapi too.

Regression Scope

Mostly around the Twitter Sync message BUT does also touch on the GQL feed stuff by changing types - the latter shouldn't really be a huge issue as any type errors should be caught at the build phase, but just pointing it out.

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

Manual, Unit (Jasmine / Karma)

Screenshots / Screen Recording

With Strapi Config

simplescreenrecorder-2023-06-09_19.24.52

Default if not set in Strapi

simplescreenrecorder-2023-06-09_19.40.24

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Ben

Merge request reports