Skip to content

Protect against multiple participation objects; and un-writeable doc. - #472

Merged
bourgeoa merged 3 commits into
mainfrom
particip
Feb 12, 2022
Merged

Protect against multiple participation objects; and un-writeable doc.#472
bourgeoa merged 3 commits into
mainfrom
particip

Conversation

@timbl

@timbl timbl commented Jan 30, 2022

Copy link
Copy Markdown
Contributor

Aimed at fixing SolidOS/chat-pane#71

Comment thread src/participation.ts Outdated
Comment thread src/participation.ts Outdated
Comment thread src/participation.ts
Comment thread src/participation.ts
Comment thread src/participation.ts Outdated
Comment thread src/participation.ts Outdated

@SharonStrats SharonStrats left a comment

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.

I tried to help with the type errors as I know you are busy. I did not test though to see if the map would work with the sort. It does solve the Typescript errors though. My system isn't setup perfectly right now, but I can try it out tomorrow. I will check to see if you already fixed it tomorrow and if not I'll see if I can test it locally.

@bourgeoa

bourgeoa commented Feb 1, 2022

Copy link
Copy Markdown
Contributor

Resolve conversations to hope make the tests run. They are blocked for one day now.

@bourgeoa
bourgeoa merged commit ea33598 into main Feb 12, 2022
@bourgeoa
bourgeoa deleted the particip branch February 12, 2022 13:57
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.

3 participants