Skip to content

added ALLOW_SECURE_COOKIES env variable & made minor changes to docs#217

Merged
thetronjohnson merged 3 commits intohoppscotch:mainfrom
SanskritiHarmukh:secure-cookies
Jul 27, 2024
Merged

added ALLOW_SECURE_COOKIES env variable & made minor changes to docs#217
thetronjohnson merged 3 commits intohoppscotch:mainfrom
SanskritiHarmukh:secure-cookies

Conversation

@SanskritiHarmukh
Copy link
Copy Markdown
Contributor

@SanskritiHarmukh SanskritiHarmukh commented Jul 24, 2024

  • Updated the environment file in Community and Enterprise Edition with new variable : ALLOW_SECURE_COOKIES
  • Changed "My workspace" to "Personal Workspace" in Workspaces page.
  • Edited the description for Buttons widget, since only 2 options are available to copy the format from- HTML and Markdown.

Signed-off-by: SanskritiHarmukh <sanskritiharmukh1908@gmail.com>
@SanskritiHarmukh SanskritiHarmukh requested a review from balub July 24, 2024 20:18
Comment thread documentation/self-host/enterprise-edition/install-and-build.mdx Outdated
Comment thread documentation/self-host/community-edition/install-and-build.mdx Outdated
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.

I don't see how this change is relevant to the PR, please remove it from this PR.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Okay, I spotted it while going through docs, thus made changes in this PR itself.

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.

I would suggest mentioning this in the PR desc or someplace else. I have no context regarding this change hence don't want to approve and have something possible wrong merged.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure, I have updated the description. Will ask Kiran to review.

SanskritiHarmukh and others added 2 commits July 25, 2024 01:59
@SanskritiHarmukh SanskritiHarmukh changed the title added ALLOW_SECURE_COOKIES env variable added ALLOW_SECURE_COOKIES env variable & made minor changes to docs Jul 24, 2024
@thetronjohnson thetronjohnson merged commit acce4c4 into hoppscotch:main Jul 27, 2024
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.

3 participants