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

feat: Story map collaboration #1133

Merged
merged 85 commits into from
Sep 11, 2023
Merged

feat: Story map collaboration #1133

merged 85 commits into from
Sep 11, 2023

Conversation

josebui
Copy link
Contributor

@josebui josebui commented Aug 22, 2023 •
edited
Loading

Description

  • Added Share dialog
  • Added Story map collaboration pending state
  • Added support for invites for not registered users

Checklist

  • Corresponding issue has been opened
  • New tests added

Related Issues

Fixes #1037
Fixes techmatters/terraso-product#336
Fixes techmatters/terraso-backend#794
Fixes #1164
Fixes #1163
Fixes #1161

@josebui josebui self-assigned this Aug 22, 2023
@josebui josebui force-pushed the feat/story-map-collaboration branch from 874e1c3 to 7a6bd77 Compare August 22, 2023 20:20
@josebui josebui force-pushed the feat/story-map-collaboration-pending-state branch from 5a2a7d6 to 65383aa Compare August 22, 2023 20:30
@josebui josebui force-pushed the feat/story-map-collaboration branch from 7a6bd77 to 4f9236a Compare August 22, 2023 20:47
@josebui josebui force-pushed the feat/story-map-collaboration-pending-state branch from 65383aa to 0716d62 Compare August 22, 2023 20:53
@josebui josebui changed the base branch from feat/story-map-collaboration to main August 29, 2023 21:27
@josebui josebui changed the title feat: Story map collaboration pending state feat: Story map collaboration Aug 29, 2023
@josebui josebui force-pushed the feat/story-map-collaboration-pending-state branch from 592cd34 to 9120e1a Compare August 29, 2023 21:39
@josebui josebui marked this pull request as ready for review August 30, 2023 17:54
paulschreiber
paulschreiber requested changes Aug 30, 2023
View reviewed changes
src/collaboration/components/MemberName.js Show resolved Hide resolved
src/collaboration/collaborationConstants.js Show resolved Hide resolved
src/collaboration/components/MembershipsList.js Show resolved Hide resolved
src/tests/uiUtils.js Show resolved Hide resolved
src/tests/apiUtils.js Show resolved Hide resolved
src/localization/i18n.js Outdated Show resolved Hide resolved
src/localization/locales/en-US.json Outdated Show resolved Hide resolved
src/storyMap/components/StoryMapForm/ShareDialog.js Show resolved Hide resolved
src/common/components/UserEmailAutocomplete.js Outdated Show resolved Hide resolved
paulschreiber
paulschreiber requested changes Sep 5, 2023
View reviewed changes
src/localization/i18n.js Outdated Show resolved Hide resolved
@paulschreiber paulschreiber force-pushed the feat/story-map-collaboration-pending-state branch from 8f2e380 to 0191a8e Compare September 5, 2023 16:10
@josebui josebui mentioned this pull request Sep 5, 2023
Closed
@josebui josebui force-pushed the feat/story-map-collaboration-pending-state branch from cb22376 to 8aafec9 Compare September 6, 2023 21:58
@josebui josebui merged commit f54fef8 into main Sep 11, 2023
8 checks passed
@josebui josebui deleted the feat/story-map-collaboration-pending-state branch September 11, 2023 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shrouxm shrouxm shrouxm left review comments

@paulschreiber paulschreiber paulschreiber approved these changes

Assignees

@josebui josebui

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@josebui @paulschreiber @shrouxm