-
Notifications
You must be signed in to change notification settings - Fork 125
chore: Fixed docker URI for kafka image #1066
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
Merged
Merged
+1
−1
Conversation
This file contains hidden or 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
Assem-Uber
approved these changes
Nov 5, 2025
Signed-off-by: Tim Chan <[email protected]>
b198ba2 to
c5f17d4
Compare
Assem-Uber
pushed a commit
to Assem-Uber/cadence-web
that referenced
this pull request
Nov 7, 2025
Signed-off-by: Tim Chan <[email protected]>
Assem-Uber
added a commit
that referenced
this pull request
Nov 24, 2025
* Create fetcher utility Signed-off-by: Assem Hafez <[email protected]> * rename query Signed-off-by: Assem Hafez <[email protected]> * Create hook for fetching history Signed-off-by: Assem Hafez <[email protected]> * add configurable throttleMs to the hook Signed-off-by: Assem Hafez <[email protected]> * use fetcher in workflow history * remove useKeepLoadingEvents Signed-off-by: Assem Hafez <[email protected]> * grouping utility Signed-off-by: Assem Hafez <[email protected]> * update test cases Signed-off-by: Assem Hafez <[email protected]> * change the api of onChange and add destroy method Signed-off-by: Assem Hafez <[email protected]> * replace getGroups with getState Signed-off-by: Assem Hafez <[email protected]> * init grouper hook Signed-off-by: Assem Hafez <[email protected]> * user grouper in history Signed-off-by: Assem Hafez <[email protected]> * update fetcher based on feedback Signed-off-by: Assem Hafez <[email protected]> * rename unmout to destroy Signed-off-by: Assem Hafez <[email protected]> * Create hook for fetching history Signed-off-by: Assem Hafez <[email protected]> * move condition into executeImmediately Signed-off-by: Assem Hafez <[email protected]> * update destroy in method Signed-off-by: Assem Hafez <[email protected]> * mock grouper throttle Signed-off-by: Assem Hafez <[email protected]> * fix use fetcher test case Signed-off-by: Assem Hafez <[email protected]> * feat: Grouped events table header (#1059) * Grouped events header Signed-off-by: Assem Hafez <[email protected]> * add test cases Signed-off-by: Assem Hafez <[email protected]> * fix comment typo Signed-off-by: Assem Hafez <[email protected]> * add space for reset button Signed-off-by: Assem Hafez <[email protected]> * move grouped table Signed-off-by: Assem Hafez <[email protected]> * Update src/views/workflow-history-v2/workflow-history-grouped-table/workflow-history-grouped-table.tsx Co-authored-by: Adhitya Mamallan <[email protected]> * Change Id to ID Signed-off-by: Assem Hafez <[email protected]> --------- Signed-off-by: Assem Hafez <[email protected]> Co-authored-by: Adhitya Mamallan <[email protected]> * chore: Fixed docker URI for kafka image (#1066) Signed-off-by: Tim Chan <[email protected]> * chore: New feature flag for Cron List View (#1068) Signed-off-by: Tim Chan <[email protected]> * use fetcher in workflow history * fix content clicks Signed-off-by: Assem Hafez <[email protected]> * fix use initial selected event Signed-off-by: Assem Hafez <[email protected]> * increase page size Signed-off-by: Assem Hafez <[email protected]> * optimize first page processing Signed-off-by: Assem Hafez <[email protected]> * fix test cases Signed-off-by: Assem Hafez <[email protected]> * fix: History content negative z-index disallow clicks (#1069) * fix content clicks Signed-off-by: Assem Hafez <[email protected]> * Update src/views/workflow-history/workflow-history.styles.ts Co-authored-by: Copilot <[email protected]> --------- Signed-off-by: Assem Hafez <[email protected]> Co-authored-by: Copilot <[email protected]> * feat: Use fetcher in workflow history (#1064) * Create fetcher utility Signed-off-by: Assem Hafez <[email protected]> * rename query Signed-off-by: Assem Hafez <[email protected]> * Create hook for fetching history Signed-off-by: Assem Hafez <[email protected]> * add configurable throttleMs to the hook Signed-off-by: Assem Hafez <[email protected]> * use fetcher in workflow history * remove useKeepLoadingEvents Signed-off-by: Assem Hafez <[email protected]> * update fetcher based on feedback Signed-off-by: Assem Hafez <[email protected]> * rename unmout to destroy Signed-off-by: Assem Hafez <[email protected]> * Create hook for fetching history Signed-off-by: Assem Hafez <[email protected]> * move condition into executeImmediately Signed-off-by: Assem Hafez <[email protected]> * update destroy in method Signed-off-by: Assem Hafez <[email protected]> * fix type error with query params Signed-off-by: Assem Hafez <[email protected]> --------- Signed-off-by: Assem Hafez <[email protected]> * fix: Autocomplete for bool values (#1072) Fix workflows query autocomplete to suggest the correct tokens for boolean values Signed-off-by: Adhitya Mamallan <[email protected]> * Add feature flag for Failover History (#1070) Signed-off-by: Adhitya Mamallan <[email protected]> * Add failover history tab (#1071) Signed-off-by: Adhitya Mamallan <[email protected]> * chore: Added ts-node to dev-deps (#1074) Signed-off-by: Tim Chan <[email protected]> * feat: Hook to fetch & filter Failover History of a domain (#1075) Add hook for fetching and filtering failover history Add helper function to check if a ClusterFailover matches a given cluster attribute Add some types/constants for domain page failovers Signed-off-by: Adhitya Mamallan <[email protected]> * feat: New Cron Tab (#1078) Signed-off-by: Tim Chan <[email protected]> * feat: Failover History table (#1076) * Add table to display failover history of a domain, with support for both active-passive and active-active domains Signed-off-by: Adhitya Mamallan <[email protected]> Co-authored-by: Copilot <[email protected]> * chore: Fix console.error pollution (#1077) * chore: Fix console.error pollution Signed-off-by: Tim Chan <[email protected]> * Relocated to src/test-utils. Signed-off-by: Tim Chan <[email protected]> --------- Signed-off-by: Tim Chan <[email protected]> * Add flag for History Page v2 (#1081) Signed-off-by: Adhitya Mamallan <[email protected]> * feat: Filters for Failover History Table (#1079) Add filters for Failover History table, but only for active-active domains Create filter for Cluster Attribute Scope, which depends on domain description to suggest possible scopes Create filter for Cluster Attribute Value, which depends on selected scope to suggest possible values/names Remove (now) unused styles file from DomainPageFailovers Signed-off-by: Adhitya Mamallan <[email protected]> * feat: Create a grouping utility that only parses new events (#1065) * grouping utility Signed-off-by: Assem Hafez <[email protected]> * update test cases Signed-off-by: Assem Hafez <[email protected]> * change the api of onChange and add destroy method Signed-off-by: Assem Hafez <[email protected]> * replace getGroups with getState Signed-off-by: Assem Hafez <[email protected]> * call onchange after updating pending events Signed-off-by: Assem Hafez <[email protected]> * use sync processBatch for first page Signed-off-by: Assem Hafez <[email protected]> * Update src/views/workflow-history/helpers/workflow-history-grouper.ts Co-authored-by: Copilot <[email protected]> * Apply PR comments * Update src/views/workflow-history/helpers/workflow-history-grouper.types.ts Co-authored-by: Copilot <[email protected]> --------- Signed-off-by: Assem Hafez <[email protected]> Co-authored-by: Copilot <[email protected]> * feat: Add modal to view full failover event (#1080) * Add modal for viewing an individual failover event, which is shown when "See more" in the table is clicked Signed-off-by: Adhitya Mamallan <[email protected]> * feat: New components for Workflow History V2 (#1082) * Add new Workflow History V2 root component, which (for now) renders only a placeholder header component and placeholder tables. * Implement Workflow History Header with basic UI and sticky functionality * Add logic in WorkflowHistoryWrapper to render V2 UI based on feature flag Signed-off-by: Adhitya Mamallan <[email protected]> * update test cases * hook for history grouper * remove comments * fetcher updates * add placeholder for fetcher * update fetcher mock * fetcher start update * update todo * Apply suggestion from @Copilot Co-authored-by: Copilot <[email protected]> * fix copilot comments * remove extra comment * lint fix * change lastFlattented initial value to -1 * remove unused import * address comments --------- Signed-off-by: Assem Hafez <[email protected]> Signed-off-by: Tim Chan <[email protected]> Signed-off-by: Adhitya Mamallan <[email protected]> Co-authored-by: Adhitya Mamallan <[email protected]> Co-authored-by: Tim Chan <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Adhitya Mamallan <[email protected]>
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.
The docker URI for kafka was recently broken.
More info: https://community.broadcom.com/tanzu/blogs/beltran-rueda-borrego/2025/08/18/how-to-prepare-for-the-bitnami-changes-coming-soon
The PR applies the same fix applied to cadence repo: https://github.com/cadence-workflow/cadence/pull/7219/files
Testing: