Skip to content

fix: remove requirement for signature token#16151

Merged
emrysal merged 1 commit intomainfrom
fix/singature_token_required
Aug 9, 2024
Merged

fix: remove requirement for signature token#16151
emrysal merged 1 commit intomainfrom
fix/singature_token_required

Conversation

@sean-brydon
Copy link
Copy Markdown
Member

What does this PR do?

Removes the hard crash if a signature token is not set with a valid license key.

How to test?

  1. Set a valid license key
  2. Remove signature token
  3. Ensure you can make bookings and also ensure EE features are present.

@graphite-app graphite-app Bot requested a review from a team August 9, 2024 12:59
@keithwillcode keithwillcode added consumer core area: core, team members only labels Aug 9, 2024
@dosubot dosubot Bot added the enterprise area: enterprise, audit log, organisation, SAML, SSO label Aug 9, 2024
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Aug 9, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (08/09/24)

1 reviewer was added to this PR based on Keith Williams's automation.

Copy link
Copy Markdown
Contributor

@emrysal emrysal left a comment

Choose a reason for hiding this comment

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

Logic is sound.

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

Labels

consumer core area: core, team members only enterprise area: enterprise, audit log, organisation, SAML, SSO ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants