Skip to content

add initial default platform key while creating a new organization#63

Merged
hari-kuriakose merged 6 commits into
mainfrom
SetInitialDefaultPlatformKey
Mar 11, 2024
Merged

add initial default platform key while creating a new organization#63
hari-kuriakose merged 6 commits into
mainfrom
SetInitialDefaultPlatformKey

Conversation

@muhammad-ali-e

Copy link
Copy Markdown
Contributor

What

  • Introduce the addition of an initial default platform key during the creation of a new organization.
  • Handled mypy issues in RemoveAllowHeaderMiddleware

Why

  • Streamlining the initial setup process for users by automatically including a default platform key, enhancing ease of use and expediting the onboarding experience.

How

  • Implementation involves the creation of a platform key at the organization's inception, ensuring a smoother initial setup for users.

Checklist

I have read and understood the Contribution Guidelines.

@chandrasekharan-zipstack chandrasekharan-zipstack left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, just left a doubt regarding connection.tenant

Comment thread backend/middleware/remove_allow_header.py
Comment thread backend/platform_settings/platform_auth_service.py

@chandrasekharan-zipstack chandrasekharan-zipstack left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@hari-kuriakose
hari-kuriakose merged commit 26a6e65 into main Mar 11, 2024
@hari-kuriakose
hari-kuriakose deleted the SetInitialDefaultPlatformKey branch March 11, 2024 14:23
pk-zipstack pushed a commit that referenced this pull request Aug 20, 2025
Co-authored-by: Hari John Kuriakose <hari@zipstack.com>
Co-authored-by: Neha <115609453+nehabagdia@users.noreply.github.com>
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.

4 participants