Skip to content

Conversation

rsergeenko
Copy link
Contributor

No description provided.

@rsergeenko rsergeenko self-assigned this May 23, 2022
…eate_consumer_groups

# Conflicts:
#	redisinsight/ui/src/constants/api.ts
#	redisinsight/ui/src/slices/browser/stream.ts
#	redisinsight/ui/src/slices/interfaces/stream.ts
@rsergeenko rsergeenko changed the base branch from main to feature/RI-2932_Consumer_Groups May 23, 2022 11:29
@rsergeenko rsergeenko requested a review from egor-zalenski May 23, 2022 11:30
}

// Asynchronous thunk action
export function addNewGroupAction(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add unit tests pls

export const validateField = (text: string) => text.replace(/\s/g, '')

export const validateEntryId = (initValue: string) => initValue.replace(/[^0-9-*]+/gi, '')
export const validateConsumerGroupId = (initValue: string) => initValue.replace(/[^0-9-$]+/gi, '')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add unit tests pls

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@rsergeenko rsergeenko merged commit ffe5194 into feature/RI-2932_Consumer_Groups May 27, 2022
@rsergeenko rsergeenko deleted the feature/RI-810_create_consumer_groups branch October 24, 2022 12:02
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.

2 participants