[Data rearchitecture] sync with master #6096
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- a new mailer + new template - a new method in BlockUserAlert to mail to a set of concerned people which replace the current one in blocked_user_monitor.rb - the corresponding spec - new line in preview mailer
Refactor error handling in ArticleStatusManager to use ApiErrorHandling consistently
…mpaign-table-appears-when-closing-modals Fix campaigns list rendering above recent news section issue on index page when navigating back via cancel button
…rror-logging-course-update Extend error logging for course data updates
- Deleted the unused ReviewerLink component. - Removed the commented-out reference to ReviewerLink in the AssignmentsLinks component.
…remove-unused-reviewerlink chore: Remove unused ReviewerLink component and cleanup related code
…ck modal when a course has no rating
…nt to be available before interacting with it
…al is visible and the ok button rendered before interacting with it
…outreachy/improve-test-suite Fix: failing test for assigned_article_spec
…or/student-view-navigation-course-page Refactor components to use navigation utility hook for student details view
…n#6061) --------- Co-authored-by: bhushan354 <[email protected]>
This feature isn't compatible with removing the Revisions table
…fix/account_request_spec Fix: failing test in account_requests_spec
This is necessary as part of the project to deprecate the Revisions table.
…cyrillefr/WikiEduDashboard into cyrillefr-Improve_BlockedUserAlert_emails_edit
…the default, to ensure all data is fetched
We have new PCORI reviews, and the links for all of them have changed.
I overlooked this change when I reimplemented the articleDetails to be completely client-side.
…ulate_logs [FIX]: Unnecessary Logs, Preventing System Inefficiencies and Inefficient Retries While Populating Dashboard Database
…i page ID - Added "crossCheckArticleTitle" function to verify and update article titles and URLs using the MediaWiki API based on the page ID. - Introduced "UPDATE_ARTICLE_TITLE_AND_URL" action in the reducer to update article titles and URLs in the Redux store when article title is changed. - Updated "fetchArticleDetails" to integrate "crossCheckArticleTitle" for dynamic title validation and updates before if fetching article first and last revision fails. - Enhanced handling of moved articles by verifying and updating titles via MediaWiki API by using "crossCheckArticleTitle" in ArticleViewer.
…ffviewer-handle-moved-articles fix(DiffViewer & ArticleViewer): handle moved articles using MediaWik…
…tudentProgramEndpoint Add student program endpoint
…ducationFoundation#6028) * Fix: Improve meeting date extraction and validation in Week component * Refactored the regex into a named-component * changed the date extraction to the course_date_utils.js * fixed the linting errors * Fix issue with meeting display after the end date * fix rspec test * fix rspec tests and check if meetings go beyond timeline.end
…reen-reader-weekdaypicker-accessibility Improve/Fix Accessibility for WeekdayPicker Component for Screen reader
…he dashboard relies on
…inor formatting edits/ corrections
…ove-documentation-phabT375642 Improve Documentation: Dashboard Deployment on Toolforge and Cloud VPS
…fix/front-end-bug Fix: front end bugs
…-to-admin-guide Add links to admin_guide.md
…check for it (WikiEducationFoundation#6085) * move StringConcatenation rule to 'permanent exceptions' section
…fix/multiwiki_student_role_spec Fix: failure in Student_role_spec
…re-sync-with-master
|
There is one spec failing but I don't think this is related to my changes because it was failing recently in other PRs too. |
gabina
merged commit ad701f7
into
WikiEducationFoundation:data-rearchitecture-for-dashboard
1 check failed
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR does
Periodic sync with master. Automatic merge failed and I manually modified
app/views/courses/articles.json.jbuilderOpen questions and concerns
< anything you learned that you want to share, or questions you're wondering about related to this PR >