- Issue created by @DamienMcKenna
- Assigned to arunkumark
- Merge request !6317Combined the exception together to common exception → (Closed) created by arunkumark
- Issue was unassigned.
- Status changed to Needs review
8 months ago 10:37am 25 January 2024 - Status changed to Needs work
8 months ago 2:17pm 25 January 2024 - 🇺🇸United States smustgrave
@arunkumark this was tagged for novice users to get started. Based on your post history believe that does not include you. Please refrain from novice issues in the future.
For the fix I think we should still keep the exception types but can still be 1 catch.
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
This needs to use a union, not remove the typehints - thanks
- First commit to issue fork.
- Status changed to Needs review
8 months ago 2:20pm 14 February 2024 - 🇲🇽Mexico GafgarionMorua Puerto Vallarta
Refactor multiple catch blocks into union type catch block.
- Status changed to RTBC
8 months ago 2:31pm 14 February 2024 - Status changed to Needs work
8 months ago 2:39pm 14 February 2024 - Status changed to Needs review
8 months ago 3:21pm 14 February 2024 - 🇲🇽Mexico GafgarionMorua Puerto Vallarta
My apologies; I have fixed the issues flagged by the pipelines. The MR is ready for review.
- Status changed to RTBC
8 months ago 3:49pm 14 February 2024 - 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
larowlan → changed the visibility of the branch 3421206-incorrect-docs-for to hidden.
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Crediting reviews, @DamienMcKenna for opening the issue and @smustgrave for mentoring
-
larowlan →
committed 41d193f7 on 10.2.x
Issue #3416798 by GafgarionMorua, arunkumark, larowlan, smustgrave,...
-
larowlan →
committed 41d193f7 on 10.2.x
-
larowlan →
committed 54f3e98a on 11.x
Issue #3416798 by GafgarionMorua, arunkumark, larowlan, smustgrave,...
-
larowlan →
committed 54f3e98a on 11.x
- Status changed to Fixed
8 months ago 11:49pm 14 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.
