Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Adding changes for persist Chat and also a change in Patch for … #339

Merged
merged 2 commits into from
Apr 8, 2024

Conversation

vinya-jag
Copy link

Changes for Persist Chat (Addition) into
Create Meetings, Update, and Fetch
Small edit into Patch Request for meeting in this page
https://docs.dyte.io/api?v=v2#/operations/editWebhook
…meeting.chatSynced

Checks done:
No errors on Swagger Editor

Source:
StopLight vinayEdit fork.
Have cleaned up this Fork of all Swagger Misalignments, and made it fully useable.
Please us this for any future edits.

Description

Describe your PR here.

Resolved issues

Closes #1

Before submitting the PR, please take the following into consideration

  • It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. If you don't have an issue, please create one.
  • Prefix your PR title with feat: , fix: , chore: , docs:, or refactor:.
  • The description should clearly illustrate what problems it solves.
  • Ensure that the commit messages follow our guidelines.
  • [ None] Resolve merge conflicts (if any).
  • [ Yes] Make sure that the current branch is upto date with the main branch.

type: boolean
x-stoplight:
id: 1rdw9emd9qk8y
description: 'If a meeting is updated to persist_chat, meeting chat would remain for a week within the meeting space'
Copy link
Member

Choose a reason for hiding this comment

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

Any specific reason for changing these to single quotes? I think stoplight generates double quotes automatically.

@@ -4942,7 +4946,7 @@ components:
type: boolean
x-stoplight:
id: k1f2cjsd28631
description: "Controls whether recordings are uploaded to Dyte's bucket. If set to false, `download_url`, `audio_download_url`, `download_url_expiry` won't be generated for a recording."
description: 'Controls whether recordings are uploaded to Dyte bucket. If set to false, `download_url`, `audio_download_url`, `download_url_expiry` won''t be generated for a recording.'
Copy link
Member

Choose a reason for hiding this comment

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

The quote in won't will be missing with this change, we'll need to escape it \'

Copy link
Author

@vinya-jag vinya-jag left a comment

Choose a reason for hiding this comment

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

Fixed Full Stop at 3 places, and Dyte's at 2 places

Copy link
Member

@roerohan roerohan left a comment

Choose a reason for hiding this comment

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

LGTM

@roerohan roerohan merged commit d3ba1db into dyte-io:main Apr 8, 2024
1 check passed
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