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.

Sign up for GitHub

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

Jump to bottom

Sprint 31 #2421

Merged
merged 222 commits into from
Dec 25, 2023
Merged

Sprint 31 #2421

merged 222 commits into from
Dec 25, 2023

Conversation

pvm-code
Copy link
Collaborator

  • PR title equals to the ticket name
  • I added the ticket to the Development section of this PR.

What was changed?

  • [ ]

roienatan and others added 30 commits November 29, 2023 22:33
@roienatan
feed scroller ; disable html globall scroller
0c2e0e8
@roienatan
custom-tab: added user-select: none to prevent selecting text possibl…
ef7fa74 
…y causing to a tab to not be clicked
@roienatan
added also -webkit-user-select
b406fee
@roienatan
try onContextMenu and useLongPress
0ed1c5a
@budnik9
fix feed item duplication in the inbox page
72365f7
@roienatan
Merge branch 'dev' into cw-2350-tabs-layout-click-issue
510f515
@roienatan
Merge branch 'dev' into cw-2334-correct-feed-scroller-location
7bfe24f
@roienatan
use useLockedBody to lock common feed and toggle scrolling for Modal
dce1b9d
@roienatan
use useLockedBody in <SplitView />
ed51abc
@budnik9
Merge pull request #2368 from daostack/cw-2334-correct-feed-scroller-…
c762692 
…location

Fix space+inbox feed scroller location #2334
@andreymikhadyuk
add check for empty value in cache transform
cb44ae8
@andreymikhadyuk
return rest cache items and add clearing of wrong data in them
a0d8f4f
@andreymikhadyuk
fix getCachedCommonById
04272eb
@andreymikhadyuk
add check for existing proposal in the subscription
dbcbe1e
@andreymikhadyuk
add log for discussion in subscription
2374ff2
@andreymikhadyuk
add log for discussion in subscription on data
8952edd
@andreymikhadyuk
add source log
25aff28
@andreymikhadyuk
update getDiscussionById to use source argument
a964114
@andreymikhadyuk
move clearing of wrong cache data to outbound state
0496b12
@andreymikhadyuk
revert cache tranform change
797e8e8
@andreymikhadyuk
fix empty cache value check in transform
6cc6a1f
@andreymikhadyuk
update InboxItem model
a30f22b
@andreymikhadyuk
add call to server when discussion is null in firebase cache
2e305c3
@andreymikhadyuk
add method to subscribe to inbox sub-collection
72444f4
@andreymikhadyuk
add method to fetch follow feed item by id
e6e1e53
@andreymikhadyuk
add method to fetch chat channel by id
48fc94f
@andreymikhadyuk
change inbox items subscription to subscribe to inbox collection
1311291
@andreymikhadyuk
Merge branch 'dev' into feature/CW-2363-inbox-collection-usage
3a90d46
@andreymikhadyuk
Merge pull request #2376 from daostack/bugfix/CW-2357-feed-item-dupli…
bb8af96 
…cation

Duplication of inbox item with query ?item= #2357
@andreymikhadyuk
add filtering of middle fetched inbox items
37d5093
budnik9 and others added 28 commits December 19, 2023 14:17
@budnik9
Merge branch 'dev' into feature/CW-2347-notion-integration-improvements
0cbbccf
@budnik9
fix firebase import
5dfdf77
@roienatan
remove unnecessary console print
14d7df6
@andreymikhadyuk
Merge branch 'dev' into feature/CW-2089-link-stream
1824252
@andreymikhadyuk
add default color for common path
0517903
@andreymikhadyuk
Merge pull request #2397 from daostack/feature/CW-2347-notion-integra…
5e5a318 
…tion-improvements

Notion integration completions #2347
@roienatan
Merge branch 'dev' into cw-2380-support-group-message
3c2611d
@andreymikhadyuk
display arrow before last common name
adb2991
@andreymikhadyuk
fix colors of hovered tree items on mobile
a4e7958
@andreymikhadyuk
align last common name with arrow correctly
366efbe
@roienatan
seems like <FloatingPortal /> implements its own body scroll disable …
68bfe02 
…so we don't need to handle that in <ContextMenu />
@andreymikhadyuk
change linked item mark text colors
33d1e93
@pvm-code
Merge pull request #2408 from daostack/cw-2300-chat-message-rtl-handl…
8a31c77 
…e-update

Handle rtl text better in chat messages
@pvm-code
Merge pull request #2410 from daostack/cw-2399-cant-scroll-body-fix
7bbfc13 
Create\edit space\common can't scroll down DEV #2399
@roienatan
code review updates
c217334
@roienatan
Merge pull request #2385 from daostack/cw-2124-open-internal-link-in-…
a0a7da6 
…the-app

open internal links in the app #2124
@roienatan
Merge branch 'dev' into cw-2380-support-group-message
dac849b
@pvm-code
CW-2354 Fixed Delete Modal loader
3e4532b
@roienatan
Merge pull request #2386 from daostack/cw-2380-support-group-message
296f948
@budnik9
provide the custom scrollSelectionIntoView function to avoid unnecess…
9c74bed 
…ary scrolling
@budnik9
Merge branch 'dev' into bugfix/CW-2388-chat-makes-feed-items-scroll
da9d387
@pvm-code
Merge branch 'dev' into feature/CW-2089-link-stream
444d8fb
@pvm-code
Merge branch 'dev' into CW-2354-message-deletion
0665b34
@pvm-code
Merge pull request #2392 from daostack/feature/CW-2089-link-stream
72fb592 
FE add "Link stream" action #2089
@pvm-code
Merge pull request #2413 from daostack/bugfix/CW-2388-chat-makes-feed…
fbd52f4 
…-items-scroll

Feed is scrolling upon typing a chat text #2388
@pvm-code
Merge pull request #2398 from daostack/CW-2354-message-deletion
cfb7ab4 
Chat: after deleting a message it disappears only after refresh #2354
@budnik9
update timestamps only when they change to prevent endless subscribin…
d75f405 
…g and unsubscribing to inbox items
@andreymikhadyuk
Merge pull request #2420 from daostack/bugfix/CW-2416-infinite-inbox-…
854bf86 
…loading

DEV Infinite loading of the Inbox #2416
@netlify Netlify
Copy link

netlify bot commented Dec 25, 2023 •
edited
Loading

✅ Deploy Preview for preview-common ready!

Name Link
🔨 Latest commit 854bf86
🔍 Latest deploy log https://app.netlify.com/sites/preview-common/deploys/6589393c1a7d1c0008c74f12
😎 Deploy Preview https://deploy-preview-2421--preview-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pvm-code pvm-code merged commit f64fe7b into staging Dec 25, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pvm-code @roienatan @budnik9 @andreymikhadyuk