Skip to content

TSK-1046 Gmail/telegram messages collaborators TSK-1097 Inbox scroll#2928

Merged
BykhovDenis merged 2 commits into
mainfrom
channels_inbox
Apr 10, 2023
Merged

TSK-1046 Gmail/telegram messages collaborators TSK-1097 Inbox scroll#2928
BykhovDenis merged 2 commits into
mainfrom
channels_inbox

Conversation

@BykhovDenis

@BykhovDenis BykhovDenis commented Apr 10, 2023

Copy link
Copy Markdown
Member

Contribution checklist

Brief description

Checklist

  • - UI test added to added/changed functionality?
  • - Screenshot is added to PR if applicable ?
  • - Does the code work? Check whether function and logic are correct.
  • - Does Changelog.md is updated with changes?
  • - Does the translations are up to date?
  • - Does it well tested?
  • - Tested for Chrome.
  • - Tested for Safari.
  • - Go through the changed code looking for typos, TODOs, commented LOCs, debugging pieces of code, etc.
  • - Rebase your branch onto master and upstream branch
  • - Is there any redundant or duplicate code?
  • - Are required links are linked to PR?
  • - Does new code is well documented ?

Related issues

https://front.hc.engineering/workbench/platform/tracker/TSK-1046
https://front.hc.engineering/workbench/platform/tracker/TSK-1097
A list of closed updated issues

Signed-off-by: Denis Bykhov <bykhov.denis@gmail.com>
@BykhovDenis
BykhovDenis requested a review from haiodo April 10, 2023 06:44
@BykhovDenis BykhovDenis self-assigned this Apr 10, 2023
@BykhovDenis BykhovDenis changed the title TSK-1046 Gmail/telegram messages collaborators TSK-1046 Gmail/telegram messages collaborators TSK-1097 Inbox scroll Apr 10, 2023
providerQuery.query(contact.class.ChannelProvider, { _id: value.provider }, (res) => ([provider] = res))
let target: Contact | undefined
const query = createQuery()
query.query(contact.class.Contact, { _id: value.attachedTo as Ref<Contact> }, (res) => ([target] = res))

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.

Do we need $: here? Since value is passed here as export let variable?

@@ -0,0 +1,44 @@
<!--
// Copyright © 2020, 2021 Anticrm Platform Contributors.
// Copyright © 2021 Hardcore Engineering Inc.

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.

Suggested change
// Copyright © 2021 Hardcore Engineering Inc.
// Copyright © 2023 Hardcore Engineering Inc.

const result: Tx[] = []

if (uTx.operations.$inc?.items !== undefined) {
const doc = (await control.findAll(uTx.objectClass, { _id: uTx.objectId }, { limit: 1 }))[0]

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.

Should we have some standalone logic in notification server plugin? and provide this information as Mixin for Classes?

Signed-off-by: Denis Bykhov <bykhov.denis@gmail.com>
@BykhovDenis
BykhovDenis merged commit 2c714e3 into main Apr 10, 2023
@BykhovDenis
BykhovDenis deleted the channels_inbox branch April 10, 2023 09:15
MichaelUray pushed a commit to MichaelUray/huly-platform that referenced this pull request Jul 9, 2026
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