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(recording) allow highlighting meeting recording moments #10981

Merged
merged 1 commit into from
Mar 14, 2022

Conversation

quitrk
Copy link
Contributor

@quitrk quitrk commented Feb 17, 2022

No description provided.

@quitrk quitrk force-pushed the tavram/bookmark branch 2 times, most recently from 208cb46 to 07471fe Compare February 21, 2022 09:05
@quitrk quitrk changed the title feat(recording) allow bookmarking meeting recording highlights feat(recording) allow highlighting meeting recording moments Feb 21, 2022
@quitrk
Copy link
Contributor Author

quitrk commented Feb 21, 2022

jenkins please test this please

@quitrk quitrk force-pushed the tavram/bookmark branch 2 times, most recently from 1df499b to 81d136d Compare February 23, 2022 11:53
@quitrk quitrk changed the title feat(recording) allow highlighting meeting recording moments [wip] feat(recording) allow highlighting meeting recording moments Feb 28, 2022
@quitrk quitrk changed the title [wip] feat(recording) allow highlighting meeting recording moments feat(recording) allow highlighting meeting recording moments Mar 2, 2022
@quitrk quitrk force-pushed the tavram/bookmark branch from 81d136d to fef6f28 Compare March 2, 2022 14:29
@quitrk quitrk force-pushed the tavram/bookmark branch from fef6f28 to fba410e Compare March 9, 2022 13:30
Copy link
Member

@saghul saghul left a comment

Choose a reason for hiding this comment

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

LGTM. What about the mobile side?

export function _abstractMapStateToProps(state: Object) {
const isRecordingRunning = getActiveSession(state, JitsiRecordingConstants.mode.FILE);
const isButtonDisabled = isHighlightMeetingMomentDisabled(state);
const { webhookProxyUrl } = state['features/base/config'];
Copy link
Member

Choose a reason for hiding this comment

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

Oh, this one is not documented in config.js (not your fault)

@quitrk
Copy link
Contributor Author

quitrk commented Mar 14, 2022

LGTM. What about the mobile side?

coming in a future PR

@quitrk quitrk merged commit d651ecb into master Mar 14, 2022
@quitrk quitrk deleted the tavram/bookmark branch March 14, 2022 08:31
ankit-programmer pushed a commit to ankit-programmer/jitsi-meet that referenced this pull request May 7, 2022
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