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

Added generic relation cell #969

Merged
merged 5 commits into from
Jul 27, 2023
Merged

Added generic relation cell #969

merged 5 commits into from
Jul 27, 2023

Conversation

lucasbordeau
Copy link
Contributor

  • Added a generic update hook that takes a mutation in argument
  • Added a generic relation cell that displays either : CompanyPicker, UserPicker or PeoplePicker

@ergomake
Copy link

ergomake bot commented Jul 27, 2023

Hi 👋

Here's a preview environment 🚀

https://front-twentyhq-twenty-969.env.ergomake.link

Environment Summary 📑

Container Source URL
front Dockerfile https://front-twentyhq-twenty-969.env.ergomake.link
server Dockerfile https://server-twentyhq-twenty-969.env.ergomake.link
postgres Dockerfile [not exposed - internal service]

Here are your environment's logs.

For questions or comments, join Discord.

Click here to disable Ergomake.

charlesBochet
charlesBochet approved these changes Jul 27, 2023
View reviewed changes
front/src/modules/people/hooks/useUpdateEntityField.ts
@@ -0,0 +1,65 @@
import { useContext } from 'react';
Copy link
Member

Choose a reason for hiding this comment

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

move this file to ui/table

@charlesBochet charlesBochet merged commit f4b8a3d into main Jul 27, 2023
@charlesBochet charlesBochet deleted the feat/generic-relation-cell branch July 27, 2023 23:28
charlesBochet added a commit that referenced this pull request Jul 28, 2023
@gitstart-twenty @v1b3m
@RubensRafael @charlesBochet @lucasbordeau @310387 @vieiralucas
feat: Add workspace delete feature (#896)
fcdde02 
* Add workspace delete feature

Co-authored-by: v1b3m <[email protected]>

* Add fixes and refactors

Co-authored-by: v1b3m <[email protected]>

* Add more fixes

Co-authored-by: v1b3m <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>

* Add workspace delete mutation

Co-authored-by: v1b3m <[email protected]>

* Complete v1 of deletion

Co-authored-by: Benjamin Mayanja <[email protected]>

* Revert unwanted changes

Co-authored-by: Benjamin Mayanja <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Update debouce import

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Fix server e2e tests on CI #3

* Fix server e2e tests on CI #4

* Fix server e2e tests on CI #5

* Added generic relation cell (#969)

* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types

* fix: 906 edit avatar style (#923)

* fix: 906 edit avatar style

* fix: 906 add avatar size enum and mapping for font and height

* fix: 906 remove unused vars

* chore: optimize size of front docker image (#965)

* Enable to drag under New button on pipeline (#970)

* Add minor fix

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

---------

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>
Co-authored-by: Lucas Bordeau <[email protected]>
Co-authored-by: 310387 <[email protected]>
Co-authored-by: Lucas Vieira <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>
AdityaPimpalkar pushed a commit to AdityaPimpalkar/twenty that referenced this pull request Aug 3, 2023
@lucasbordeau @AdityaPimpalkar
Added generic relation cell (twentyhq#969)
448caa6 
* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types
AdityaPimpalkar pushed a commit to AdityaPimpalkar/twenty that referenced this pull request Aug 3, 2023
@gitstart-twenty @v1b3m
@RubensRafael @charlesBochet @lucasbordeau @310387 @vieiralucas @AdityaPimpalkar
feat: Add workspace delete feature (twentyhq#896)
4324259 
* Add workspace delete feature

Co-authored-by: v1b3m <[email protected]>

* Add fixes and refactors

Co-authored-by: v1b3m <[email protected]>

* Add more fixes

Co-authored-by: v1b3m <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>

* Add workspace delete mutation

Co-authored-by: v1b3m <[email protected]>

* Complete v1 of deletion

Co-authored-by: Benjamin Mayanja <[email protected]>

* Revert unwanted changes

Co-authored-by: Benjamin Mayanja <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Update debouce import

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Fix server e2e tests on CI twentyhq#3

* Fix server e2e tests on CI twentyhq#4

* Fix server e2e tests on CI twentyhq#5

* Added generic relation cell (twentyhq#969)

* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types

* fix: 906 edit avatar style (twentyhq#923)

* fix: 906 edit avatar style

* fix: 906 add avatar size enum and mapping for font and height

* fix: 906 remove unused vars

* chore: optimize size of front docker image (twentyhq#965)

* Enable to drag under New button on pipeline (twentyhq#970)

* Add minor fix

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

---------

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>
Co-authored-by: Lucas Bordeau <[email protected]>
Co-authored-by: 310387 <[email protected]>
Co-authored-by: Lucas Vieira <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>
AdityaPimpalkar pushed a commit to AdityaPimpalkar/twenty that referenced this pull request Aug 4, 2023
@lucasbordeau @AdityaPimpalkar
Added generic relation cell (twentyhq#969)
6cc165c 
* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types
AdityaPimpalkar pushed a commit to AdityaPimpalkar/twenty that referenced this pull request Aug 4, 2023
@gitstart-twenty @v1b3m
@RubensRafael @charlesBochet @lucasbordeau @310387 @vieiralucas @AdityaPimpalkar
feat: Add workspace delete feature (twentyhq#896)
cc89232 
* Add workspace delete feature

Co-authored-by: v1b3m <[email protected]>

* Add fixes and refactors

Co-authored-by: v1b3m <[email protected]>

* Add more fixes

Co-authored-by: v1b3m <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>

* Add workspace delete mutation

Co-authored-by: v1b3m <[email protected]>

* Complete v1 of deletion

Co-authored-by: Benjamin Mayanja <[email protected]>

* Revert unwanted changes

Co-authored-by: Benjamin Mayanja <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Update debouce import

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Fix server e2e tests on CI twentyhq#3

* Fix server e2e tests on CI twentyhq#4

* Fix server e2e tests on CI twentyhq#5

* Added generic relation cell (twentyhq#969)

* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types

* fix: 906 edit avatar style (twentyhq#923)

* fix: 906 edit avatar style

* fix: 906 add avatar size enum and mapping for font and height

* fix: 906 remove unused vars

* chore: optimize size of front docker image (twentyhq#965)

* Enable to drag under New button on pipeline (twentyhq#970)

* Add minor fix

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

---------

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>
Co-authored-by: Lucas Bordeau <[email protected]>
Co-authored-by: 310387 <[email protected]>
Co-authored-by: Lucas Vieira <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>
AdityaPimpalkar pushed a commit to AdityaPimpalkar/twenty that referenced this pull request Aug 4, 2023
@gitstart-twenty @AdityaPimpalkar
Add ui/progress-bar stories (twentyhq#936)
66c1432 
* Add ui/progress-bar stories

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Add requested changes

Co-authored-by: RubensRafael <[email protected]>
Co-authored-by: v1b3m <[email protected]>

---------

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

Add pull request previews (twentyhq#954)

Add ergomake

Co-authored-by: Lucas Vieira <[email protected]>

[Do not merge] Try ergomake (twentyhq#958)

* Try ergomake

* Try redeploy

Update double cell input width to be fixed (twentyhq#946)

Fix double cell input width to be fixed

Feat/improve editable cell (twentyhq#959)

* Removed isSomeInputInEditMode

* Removed console.log

* Added a first version of generic cell text

* Removed metadata from entity table  V1

* Fix

* Fix

* Fix

TWNTY-892 - Add ui/title stories (twentyhq#955)

* Add ui/title stories

Co-authored-by: RubensRafael <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Fix linter issues

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

---------

Co-authored-by: RubensRafael <[email protected]>
Co-authored-by: v1b3m <[email protected]>

Add Timeline End Icon (twentyhq#945)

* -added timeline end icon

* - fixed styledDiv in component

* - fixed icon size

Short variant for filter texts (twentyhq#943)

* - added a short variant for filter labels in the filter bar

* - fixed tests
- moved colon to shortoperand

* - fixed formatting

TWNTY-895 - Add ui/checkmark stories (twentyhq#960)

Add ui/checkmark stories

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: Thiago Nascimbeni <[email protected]>

feat: upload profile picture from google (twentyhq#964)

* feat: upload profile picture from google

* fix: only add profile picture if user don't have any

feat: increase upload size limit (twentyhq#962)

eat: create ViewField model (twentyhq#961)

* feat: create ViewField model

- Created ViewField prisma model
- Added ViewField server resolvers for findMany/updateOne
- Added getViewFields/updateViewField graphql queries

Closes twentyhq#849

* chore: update node version in .nvmrc files

feat: implement e2e test for CompanyResolver (twentyhq#944)

* feat: wip e2e server test

* feat: use github action postgres & use infra for local

* feat: company e2e test

* feat: add company e2e test for permissions

* Simplify server e2e test run

* Fix lint

---------

Co-authored-by: Charles Bochet <[email protected]>

Fix server e2e tests on CI twentyhq#1

Fix server e2e tests on CI twentyhq#2

Fix server e2e tests on CI twentyhq#3

Fix server e2e tests on CI twentyhq#4

Fix server e2e tests on CI twentyhq#5

Added generic relation cell (twentyhq#969)

* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types

fix: 906 edit avatar style (twentyhq#923)

* fix: 906 edit avatar style

* fix: 906 add avatar size enum and mapping for font and height

* fix: 906 remove unused vars

chore: optimize size of front docker image (twentyhq#965)

Enable to drag under New button on pipeline (twentyhq#970)

feat: Add workspace delete feature (twentyhq#896)

* Add workspace delete feature

Co-authored-by: v1b3m <[email protected]>

* Add fixes and refactors

Co-authored-by: v1b3m <[email protected]>

* Add more fixes

Co-authored-by: v1b3m <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>

* Add workspace delete mutation

Co-authored-by: v1b3m <[email protected]>

* Complete v1 of deletion

Co-authored-by: Benjamin Mayanja <[email protected]>

* Revert unwanted changes

Co-authored-by: Benjamin Mayanja <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Update debouce import

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Fix server e2e tests on CI twentyhq#3

* Fix server e2e tests on CI twentyhq#4

* Fix server e2e tests on CI twentyhq#5

* Added generic relation cell (twentyhq#969)

* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types

* fix: 906 edit avatar style (twentyhq#923)

* fix: 906 edit avatar style

* fix: 906 add avatar size enum and mapping for font and height

* fix: 906 remove unused vars

* chore: optimize size of front docker image (twentyhq#965)

* Enable to drag under New button on pipeline (twentyhq#970)

* Add minor fix

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

---------

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>
Co-authored-by: Lucas Bordeau <[email protected]>
Co-authored-by: 310387 <[email protected]>
Co-authored-by: Lucas Vieira <[email protected]>
Co-authored-by: Charles Bochet <[email protected]>

feat: rename comment thread into activity (twentyhq#939)

* feat: rename commentThread into activity server

* feat: rename commentThread into activity front

* feat: migration only create tables

feat: migration only create tables

* Update activities

* fix: rebase partial fix

* fix: all rebase problems and drop activity target alter

* fix: lint

* Update migration

* Update migration

* Fix conflicts

* Fix conflicts

---------

Co-authored-by: Charles Bochet <[email protected]>

Fix storybook tests

fix: front not running properly (twentyhq#971)

docs: add DatePicker and ImageInput stories (twentyhq#980)

Closes twentyhq#979

feat: search activities (twentyhq#972)

feat: I can delete my account easily (twentyhq#977)

* Add support for account deletion

Co-authored-by: v1b3m <[email protected]>

* Add more fixes

Co-authored-by: Benjamin Mayanja <[email protected]>

* Add more fixes

Co-authored-by: v1b3m <[email protected]>

---------

Co-authored-by: v1b3m <[email protected]>

Feat/generic editable cell chip (twentyhq#982)

* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types

* wip

* Renamed to view field

* Use new view field structure to have chip working

* Finished

* Added a temp feature flag

chore: Add ui/modal stories (twentyhq#967)

* Add ui/modal stories

Co-authored-by: Matheus <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: Matheus <[email protected]>

* Fix the formatting

Co-authored-by: Benjamin Mayanja <[email protected]>
Co-authored-by: Matheus <[email protected]>

---------

Co-authored-by: Matheus <[email protected]>
Co-authored-by: v1b3m <[email protected]>

chore: Add ui/tooltip stories (twentyhq#966)

* Add ui/tooltip stories

Co-authored-by: Thiago Nascimbeni <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: Thiago Nascimbeni <[email protected]>

* Fix linting

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: Thiago Nascimbeni <[email protected]>

---------

Co-authored-by: Thiago Nascimbeni <[email protected]>
Co-authored-by: v1b3m <[email protected]>

Various fixes on table, board, tasks (twentyhq#983)

* Misc fixes

* Misc fixes

* Misc fixes

* Fix login

Add profile pictures to people and fix account/workspace deletion (twentyhq#984)

* Fix LinkedIn URL not redirecting to the right url

* add avatars for people and seeds

* Fix delete account/workspace

* Add people picture on other pages

* Change style of delete button

* Revert modal to previous size

* Fix tests

Remove empty values from relation picker (twentyhq#986)

fix: empty name values in SingleEntitySelectBase

Feat/generic editable cell all types (twentyhq#987)

* Added generic relation cell

* Deactivated debug

* Added default warning

* Put back display component

* Removed unused types

* wip

* Renamed to view field

* Use new view field structure to have chip working

* Finished

* Added a temp feature flag

* Added double text chip cell

* Ok

* Finished tables

* Fixed icon size

* Fixed bug on date field

* Use icon index

* Fix

* Fixed naming

* Fix

* removed file from merge

* Fixed tests

* Coverage

Add dueDate and assignee on notes (twentyhq#988)

* Add dueDate and assignee on notes

* Fix tests

* Fix tests

Add fake characters to prevent password managers from filling fields (twentyhq#989)

Add ability to properly cast a string, number, null to an integer (twentyhq#990)

Inbox task 2 (twentyhq#991)

* Add ability to properly cast a string, number, null to an integer

* Adding Tab UI component

* Only trigger chromatic when asked

docs: fix instructions to start projects in the docker setup flow (twentyhq#995)

Fix margin on DeleteModal overlay (twentyhq#998)

* Fix margin on DeleteModal overlay

* Update chromatic ci triggers

* Update chromatic ci triggers

Changes to commands in search window (twentyhq#996)

- changes to commands in search window

style: update deactivated state styles in onboarding button (twentyhq#997)

[952] style: update deactivated state styles in onboarding button

Clean and re-organize post table refactoring (twentyhq#1000)

* Clean and re-organize post table refactoring

* Fix tests

feat: add column resizing (twentyhq#975)

* feat: add column resizing

Closes twentyhq#817

* Use mouse up and down instead of dragging

---------

Co-authored-by: Charles Bochet <[email protected]>

Fix table mock mode (twentyhq#1007)

Add deploy buttons and clean environment variables (twentyhq#974)

* add render.yaml

* Clean environment variables

---------

Co-authored-by: Charles Bochet <[email protected]>

feat: implement user impersonation feature (twentyhq#976)

* feat: wip impersonate user

* feat: add ability to impersonate an user

* fix: remove console.log

* fix: unused import

Scroll to currently softfocus cell (twentyhq#1008)

* - scroll to currently softfocus cell

* - moved useEffect to CellSoftFocus component

Add tasks page (twentyhq#1015)

* Refactor top bar component

* Add task page with tabs

* Add tasks

* Add logic for task status

* Fix isoweek definition

* Enable click on task

* Deduplicate component

* Lint

---------

Co-authored-by: Charles Bochet <[email protected]>

A few polish on tasks (twentyhq#1023)

A few polishing on tasks

Update chromatic CI trigger

Update chromatic CI trigger

Rename SIGN_IN_PREFILLED env variable

Fix env variables for dev setup (twentyhq#1031)

[server] set local setup as default in .env.example (twentyhq#1045)

In storybook, I see a ButtonIconGroup component (twentyhq#1039)

Add ButtonIconGroup storybook components

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

Move trash icon to the top bar of right drawer (twentyhq#1014)

* Move trash icon to the top bar of right drawer

Co-authored-by: Matheus <[email protected]>

* Fix background

Co-authored-by: Matheus <[email protected]>

* Refactor the code

Co-authored-by: Matheus <[email protected]>

---------

Co-authored-by: Matheus <[email protected]>

Fix hotkeys for tasks page (twentyhq#1034)

- fix hotkeys for tasks

Fix twentyhq#1038: Logout button should be change to gray (twentyhq#1052)

feat: persist resized column widths (twentyhq#1017)

* feat: persist resized column widths

Closes twentyhq#981

* test: mock company and person view fields

Closes twentyhq#710: Add the number of opportunities on each stage (twentyhq#1011)

* Add the number of oppurtunities on each stage

* Remove excess css properties in Boardcolumn.tsx and use theme

* Remove padding from oppurnities counters

Sanitize url before fetching favicon and display letter avatar if it can't be retrieved (twentyhq#1035)

* Sanitize url before fetching favicon and display letter avatar if it can't be retrieved

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Priorotise www for apple.com domain

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Add requested changes

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Fix the tests

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

* Change avatar generation strategy

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

---------

Co-authored-by: v1b3m <[email protected]>
Co-authored-by: RubensRafael <[email protected]>

Fix: positioning of label to rename columns (twentyhq#1051)

Fix positioning of label to rename columns

Co-authored-by: Mael FOSSO <[email protected]>

Fix Avatars no centered (twentyhq#1050)

- fix workspace icon now centered
- fix profile avatar now centered

Feat/filter activity inbox (twentyhq#1032)

* Move files

* Add filtering for tasks inbox

* Add filter dropdown for single entity

* Minor

* Fill empty button

* Refine logic for filter dropdown

* remove log

* Fix unwanted change

* Set current user as default filter

* Add avatar on filter

* Improve initialization of assignee filter

* Add story for Tasks page

* Add more stories

* Add sotry with no tasks

* Improve dates

* Enh tests

---------

Co-authored-by: Charles Bochet <[email protected]>

Refactor: Morph strategy on `PipelineProgress` (twentyhq#1065)

* Deprecate pipelineprogress backref on person to improve naming

* Remove deprecated point of contact fields

* Add company and person entities on pipelineprogress

* Migrate data from old progressable to new entity fields

* Codegen frontend

* Use company Id, deprecate progressableId

* Get rid of deprecated progressableId field

* Remove deprecated progressableType field from pipeline progress

* Remove useless migrations

Improve mouse tracking (twentyhq#1061)

* Improve mouse tracking

* Fix lint

* Fix regression on Filters

* Fix according to review

[PeoplePicker] fix missing avatar in user search dropdown (twentyhq#1069)

Fix pipeline performance issue (twentyhq#1070)

Remove constraint activity morph relations foreign keys

Adding the favorite button

favorites schema

favorite ability handler

favorite module export

front end UI

front end graphql additions

server ability handlers

server resolvers and services

commits squashed and merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@charlesBochet charlesBochet charlesBochet approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lucasbordeau @charlesBochet