Skip to content

[PB-6107]: feature/override file versioning for a given user#990

Open
xabg2 wants to merge 1 commit intomasterfrom
feature/enable-file-versioning-activation-from-payments
Open

[PB-6107]: feature/override file versioning for a given user#990
xabg2 wants to merge 1 commit intomasterfrom
feature/enable-file-versioning-activation-from-payments

Conversation

@xabg2
Copy link
Contributor

@xabg2 xabg2 commented Mar 12, 2026

Enable file versioning from the gateway for a given user with the tierId.

@xabg2 xabg2 self-assigned this Mar 12, 2026
@xabg2 xabg2 added the enhancement New feature or request label Mar 12, 2026
@sonarqubecloud
Copy link

@xabg2 xabg2 marked this pull request as ready for review March 12, 2026 12:01
Comment on lines +425 to +436
if (label === LimitLabels['FileVersionEnabled']) {
if (!tierId) {
throw new BadRequestException(
`tierId is required for feature '${externalLimitName}'`,
);
}

const limit = await this.limitsRepository.findLimitByLabelAndTier(
tierId,
label,
);

Copy link
Collaborator

Choose a reason for hiding this comment

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

It would be good if we can provide the support team with something like: "choose the feature you want to set for the user" and give them multiple options that are fetched from this API so you do not need to provide the tierId to do things like this.

Example:

  1. You create an endpoint to fetch some limits with their id, label and value
  2. Use that endpoint to fetch data for the support bot and give the support team the option to choose between all the limits fetched
  3. The support team chooses one of the limits
  4. You have already de limit id and label so you can just set the overridden limit here.

(just and idea for the future) @xabg2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ready-for-preview

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants