Chat constants reload#2455
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Moving back to draft until the library is ready for release - but please still review it! |
e2f0ab3 to
29dbdb7
Compare
|
Im putting together an integration branch for everything related to chat-js 0.5.0 to save endless rebasing, please review :) |
Fix formatting for links
…cs-broken Fix bug where support link goes to /docs/support
Local implementation of compile-time feature flags like we added in ably/ably-ui#650
[WEB-4265] Add support for Ably Insights
Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.24.0 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…ers-7.26.10 chore(deps): bump @babel/helpers from 7.24.0 to 7.26.10
vladvelici
left a comment
There was a problem hiding this comment.
👍
Just a minor comment
29dbdb7 to
ae50b58
Compare
* Fix incorrect links in content docs * Fix bug where support link goes to /docs/support * feat: initial ably-ui insights setup * refactor: consolidate SessionState types into a single one * cleanup: externalScriptsData doesn't change at runtime * feat: compile-time feature flags like ably-ui Local implementation of compile-time feature flags like we added in ably/ably-ui#650 * cleanup: remove unused GTM trackPageView function * chore(deps): bump @babel/helpers from 7.24.0 to 7.26.10 Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.24.0 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * chat/js: rename DefaultRoomOptions to AllFeaturesEnabled * chat/js: make explicit reference to client options --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Fix incorrect links in content docs * Fix bug where support link goes to /docs/support * feat: initial ably-ui insights setup * refactor: consolidate SessionState types into a single one * cleanup: externalScriptsData doesn't change at runtime * feat: compile-time feature flags like ably-ui Local implementation of compile-time feature flags like we added in ably/ably-ui#650 * cleanup: remove unused GTM trackPageView function * chore(deps): bump @babel/helpers from 7.24.0 to 7.26.10 Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.24.0 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * chat/js: rename DefaultRoomOptions to AllFeaturesEnabled * chat/js: make explicit reference to client options --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Fix incorrect links in content docs * Fix bug where support link goes to /docs/support * feat: initial ably-ui insights setup * refactor: consolidate SessionState types into a single one * cleanup: externalScriptsData doesn't change at runtime * feat: compile-time feature flags like ably-ui Local implementation of compile-time feature flags like we added in ably/ably-ui#650 * cleanup: remove unused GTM trackPageView function * chore(deps): bump @babel/helpers from 7.24.0 to 7.26.10 Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.24.0 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * chat/js: rename DefaultRoomOptions to AllFeaturesEnabled * chat/js: make explicit reference to client options --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * Chat constants reload (#2455) * Fix incorrect links in content docs * Fix bug where support link goes to /docs/support * feat: initial ably-ui insights setup * refactor: consolidate SessionState types into a single one * cleanup: externalScriptsData doesn't change at runtime * feat: compile-time feature flags like ably-ui Local implementation of compile-time feature flags like we added in ably/ably-ui#650 * cleanup: remove unused GTM trackPageView function * chore(deps): bump @babel/helpers from 7.24.0 to 7.26.10 Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.24.0 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * chat/js: rename DefaultRoomOptions to AllFeaturesEnabled * chat/js: make explicit reference to client options --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chat: bump to 0.5 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * Chat constants reload (#2455) * Fix incorrect links in content docs * Fix bug where support link goes to /docs/support * feat: initial ably-ui insights setup * refactor: consolidate SessionState types into a single one * cleanup: externalScriptsData doesn't change at runtime * feat: compile-time feature flags like ably-ui Local implementation of compile-time feature flags like we added in ably/ably-ui#650 * cleanup: remove unused GTM trackPageView function * chore(deps): bump @babel/helpers from 7.24.0 to 7.26.10 Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.24.0 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * chat/js: rename DefaultRoomOptions to AllFeaturesEnabled * chat/js: make explicit reference to client options --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chat: bump to 0.5 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * Chat constants reload (#2455) * Fix incorrect links in content docs * Fix bug where support link goes to /docs/support * feat: initial ably-ui insights setup * refactor: consolidate SessionState types into a single one * cleanup: externalScriptsData doesn't change at runtime * feat: compile-time feature flags like ably-ui Local implementation of compile-time feature flags like we added in ably/ably-ui#650 * cleanup: remove unused GTM trackPageView function * chore(deps): bump @babel/helpers from 7.24.0 to 7.26.10 Bumps [@babel/helpers](https://github.com/babel/babel/tree/HEAD/packages/babel-helpers) from 7.24.0 to 7.26.10. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.26.10/packages/babel-helpers) --- updated-dependencies: - dependency-name: "@babel/helpers" dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> * chat: update message version comparison function names (#2462) * chat: add JS message copying and update signature (#2470) * chat/js: rename DefaultRoomOptions to AllFeaturesEnabled * chat/js: make explicit reference to client options --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chat: bump to 0.5 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Andrew Bulat <andrii.bulat@gmail.com> Co-authored-by: Mark Hulbert <39801222+m-hulbert@users.noreply.github.com> Co-authored-by: Greg Holmes <iam@gregholmes.co.uk> Co-authored-by: Kenneth Kalmer <kenneth.kalmer@ably.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Context
CHA-851
JS PR: ably/ably-chat-js#478
Description
This pull request includes several updates to the
content/chat/rooms/index.textileandcontent/chat/setup.textilefiles to replace the usage ofRoomOptionsDefaultswithAllFeaturesEnabledand to addChatClientOptionsreferences for better understandability.