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

Bugfix for GitLab mirror workflow #1869

Closed
wants to merge 711 commits into from

Conversation

sonoransun
Copy link
Contributor

Description

This is to test a fix for the GitLab mirror workflow action. It was observed that manual commands used a fully specified target URL with .git suffix, which is tested here.

References: CV2-4533

How has this been tested?

These commands match the working manual commands.

Checklist

  • I have performed a self-review of my own code
  • I considered secure coding practices when writing this code. Any security concerns are noted above.
  • My changes generate no new warnings

caiosba and others added 30 commits May 27, 2022 17:44
…* reports.

For both native and imported.

Reference: CHECK-1810.
…sion" when there is no annotator.

Some old reports don't have a user attached to the report. This fails when migrating reports to fact-checks because user is mandatory for fact-checks. For those cases, fallback to the user attached to the report version.

Reference: CHECK-1959.
Some old reports don't have versions or user. For those cases, fallback to the Fetch Bot, because we can't create fact-checks without a user.

Reference: CHECK-1959.
Now the "average number of messages per day" includes the number of newsletters sent to each subscriber.

Reference: CHECK-2024.
"Messages sent" should include all types of search results: relevant, irrelevant and no feedback.

Reference: CHECK-2024.
Keep the truncation only for title and summary. We can't truncate the URL because a broken link won't take the user to the full fact-check.

Fixes CHECK-1985.
The filter by channel has not been working since we changed the "project_medias.channel" column to be a serialized array field.

Reference: CHECK-1985.
* Faster item creation by skipping some background jobs creation if they don't apply
* Require a user in order to save data
* Rate limits

Reference: CHECK-2093.
This way, it's possible to retrieve a single team task by passing its database ID.

Reference: CHECK-2089.
There are two Sidekiq queues for the tipline background jobs: "smooch" and "smooch_priority".

Reference: CHECK-2125.
…l" configuration key is updated (CHECK-2129)
Otherwise some subscribers can receive the same newsletter twice.

Fixes CHECK-2011.
caiosba and others added 25 commits February 14, 2024 21:10
…search results. (#1801)

Messenger changed the way it formats messages... now it breaks a single long message into smaller messages, so for long fact-checks, messages can be delivered out of order, which is super weird (like, a title in one message, the link in another message, etc.). This commit changes how tipline search results are delivered for Facebook Messenger: no fact-check summary, only title and URL.

Fixes CV2-4287.
… job. (#1819)

Add the workspace name to the error message so we can better handle it in Sentry (for example, archive the ones that are expected to fail).

Fixes CV2-4281.
* CV2-4351: fix sentry issue

* CV2-4352: delete old tipline requests(annotation of type smooch)

* CV2-4352: add logs to rake task
* CV2-2357: set default value for last_received_terms_email_at

* CV2-2357: fix migration
* CV2-4458: add uuid column to media

* CV2-4458: fix rake task

* CV2-4458: use downcase to get uuid
* CV2-4458: bulk-select claims to get uuid

* CV2-4458: cleanup

* CV2-4458: apply PR comments
… `TeamType.get_report` field (#1854)

* Return placeholder values for report introduction settings in GraphQL `TeamType.get_report` field.
* This way, API clients know (and can use) how the placeholder values are expected to be formatted.
* Reference: CV2-4424.
* TeamType#get_report: Return placeholders for all workspace languages, not only the ones that have report settings configured
* Add mk mn locales and use newer meedan/rails-i18n

---------

Co-authored-by: Alexandre Amorim <[email protected]>
@sonoransun sonoransun closed this Apr 26, 2024
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.

8 participants