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.

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

Jira Server: show integrated task #6092

Merged
merged 11 commits into from
Mar 4, 2022

Conversation

igorlesnenko
Copy link
Contributor

@igorlesnenko igorlesnenko commented Feb 22, 2022

Resolves #5994

Needs #6059

image

How to test:

  • Add Jira Server integration
  • Create a task in parabol
  • Using push to integration menu, push a task to the selected Jira Server
  • See the task was pushed successfully
  • Open task in Jira Server, see it was created correctly
  • Edit task in Jira Server
  • Open parabol board, see the task is loaded correctly

@igorlesnenko igorlesnenko changed the base branch from master to jira-server-create-task February 22, 2022 14:43
@igorlesnenko igorlesnenko marked this pull request as ready for review February 22, 2022 15:44
@blimmer
Copy link
Contributor

blimmer commented Feb 22, 2022

@igorlesnenko , I cherry-picked #6104 into this branch to unblock you. Just FYI in case you see merge conflicts from it.

packages/client/components/TaskIntegrationLink.tsx Outdated Show resolved Hide resolved
packages/server/dataloader/jiraServerLoaders.ts Outdated Show resolved Hide resolved
packages/server/dataloader/jiraServerLoaders.ts Outdated Show resolved Hide resolved
packages/server/graphql/types/JiraServerIssue.ts Outdated Show resolved Hide resolved
@igorlesnenko igorlesnenko merged commit 6635044 into jira-server-create-task Mar 4, 2022
@igorlesnenko igorlesnenko deleted the jira-server-show-task branch March 4, 2022 10:26
@igorlesnenko
Copy link
Contributor Author

merged into #6059 as easier to test and do changes

mattkrick added a commit that referenced this pull request Mar 25, 2022
* Add generic createTaskIntegration mutation

* Simplify

* Remove old mutations

* Update demo

* Use existing integration provider service

* authLoaderKey instead of returning data loader

* Move db actions outside

* getCreatedBySomeoneElseComment

* Remove internal state

* Add manager factory

* Tweak variable names

* Add mask false

* Allow to use any custom auth loader

* Simplify

* fix @relay(mask: false) bug

* Change CreateTaskResponse

* Change CreateTaskResponse

* Rename projectId to integrationRepoId

* Formatting

* Add Jira Server to RepoIntegrations

* Remove added id from GitHub schema

It already has an id.

* Tweak graphql.ts

* Add JiraServerTaskintegrationManager

* addComment

* Fix comment

* Add avatar fetching for Jira Server

* Fix TypeError: jiraProjects is not iterable

* Tweak type

* use useFragment instead of createFragmentContainer

* Cleanup JiraServerIntegraiton GraphQL types

* Convert fetchJiraServerProjects into a dataloader

* Improve JiraServerRestManager error reporting

* Fix JiraServerRestManager request

* Jira Server: show integrated task  (#6092)

* Jira Server: show integrated task

* Tweak import paths for circleci build

* build(server): fix jest transforms

* Tweak isTypeOf

* Tweak naming

* Do not call factory from dataloader

Co-authored-by: Ben Limmer <[email protected]>

* Fix IntegrationRepoId

* Clean up

* Fix issueKey

* Fix provider id type

* ids split - assume the input is valid and return non-null types

* Simplify

* Specific type

* Replace interface with AbstractTaskIntegrationManager

* Replace getCreatedBySomeoneElseComment with add

* Jira server - store issueId and repositoryId

* Use interface, store context and info in state

* Merge Jira Server integration manager into JiraServerRestManager

* Rename GitHub integration manager to GitHubServerManager

* Make jira integration manager similar to others

* Update JiraServer icon

Co-authored-by: Matt Krick <[email protected]>
Co-authored-by: Georg Bremer <[email protected]>
Co-authored-by: Ben Limmer <[email protected]>
igorlesnenko added a commit that referenced this pull request Mar 30, 2022
* Add generic createTaskIntegration mutation

* Simplify

* Remove old mutations

* Update demo

* Use existing integration provider service

* authLoaderKey instead of returning data loader

* Move db actions outside

* getCreatedBySomeoneElseComment

* Remove internal state

* Add manager factory

* Tweak variable names

* Add mask false

* Allow to use any custom auth loader

* Simplify

* fix @relay(mask: false) bug

* Change CreateTaskResponse

* Change CreateTaskResponse

* Rename projectId to integrationRepoId

* Formatting

* Add Jira Server to RepoIntegrations

* Remove added id from GitHub schema

It already has an id.

* Tweak graphql.ts

* Add JiraServerTaskintegrationManager

* addComment

* Fix comment

* Jira Server: show integrated task

* Tweak import paths for circleci build

* Add avatar fetching for Jira Server

* build(server): fix jest transforms

* Fix TypeError: jiraProjects is not iterable

* Tweak type

* Tweak isTypeOf

* use useFragment instead of createFragmentContainer

* Cleanup JiraServerIntegraiton GraphQL types

* Convert fetchJiraServerProjects into a dataloader

* Improve JiraServerRestManager error reporting

* feat(jira-server): List issues in sprint poker

* Fix JiraServerRestManager request

* Jira Server: show integrated task  (#6092)

* Jira Server: show integrated task

* Tweak import paths for circleci build

* build(server): fix jest transforms

* Tweak isTypeOf

* Tweak naming

* Do not call factory from dataloader

Co-authored-by: Ben Limmer <[email protected]>

* Fix IntegrationRepoId

* Clean up

* Fix issueKey

* Fix provider id type

* add jiraServer feature flag

* Remove jiraServer feature flag

* getApiManager

* Fix initManager type

* Add MockScopingList

* ids split - assume the input is valid and return non-null types

* Simplify

* Specific type

* Tweak poker tabs

* Call integration manager directly

* Remove fetch jira server projects

* Replace interface with AbstractTaskIntegrationManager

* Replace interface with AbstractTaskIntegrationManager

* Replace getCreatedBySomeoneElseComment with add

* Jira server - store issueId and repositoryId

* Use interface, store context and info in state

* Merge Jira Server integration manager into JiraServerRestManager

* Rename GitHub integration manager to GitHubServerManager

* Make jira integration manager similar to others

* Update JiraServer icon

* non-null addCreatedBySomeoneElseComment

Co-authored-by: Matt Krick <[email protected]>
Co-authored-by: Georg Bremer <[email protected]>
Co-authored-by: Ben Limmer <[email protected]>
mattkrick added a commit that referenced this pull request Apr 1, 2022
* Add generic createTaskIntegration mutation

* Simplify

* Remove old mutations

* Update demo

* Use existing integration provider service

* authLoaderKey instead of returning data loader

* Move db actions outside

* getCreatedBySomeoneElseComment

* Remove internal state

* Add manager factory

* Tweak variable names

* Add mask false

* Allow to use any custom auth loader

* Simplify

* fix @relay(mask: false) bug

* Change CreateTaskResponse

* Change CreateTaskResponse

* Rename projectId to integrationRepoId

* Formatting

* Add Jira Server to RepoIntegrations

* Remove added id from GitHub schema

It already has an id.

* Tweak graphql.ts

* Add JiraServerTaskintegrationManager

* addComment

* Fix comment

* Jira Server: show integrated task

* Tweak import paths for circleci build

* Add avatar fetching for Jira Server

* build(server): fix jest transforms

* Fix TypeError: jiraProjects is not iterable

* Tweak type

* Tweak isTypeOf

* use useFragment instead of createFragmentContainer

* Cleanup JiraServerIntegraiton GraphQL types

* Convert fetchJiraServerProjects into a dataloader

* Improve JiraServerRestManager error reporting

* feat(jira-server): List issues in sprint poker

* Fix JiraServerRestManager request

* Jira Server: show integrated task  (#6092)

* Jira Server: show integrated task

* Tweak import paths for circleci build

* build(server): fix jest transforms

* Tweak isTypeOf

* Tweak naming

* Do not call factory from dataloader

Co-authored-by: Ben Limmer <[email protected]>

* Fix IntegrationRepoId

* Clean up

* Fix issueKey

* Fix provider id type

* add jiraServer feature flag

* Remove jiraServer feature flag

* getApiManager

* Fix initManager type

* Add MockScopingList

* Allow adding a Jira Server task to poker scope

* Show Jira Server description

* ids split - assume the input is valid and return non-null types

* Simplify

* Specific type

* Tweak poker tabs

* Call integration manager directly

* Remove fetch jira server projects

* Replace interface with AbstractTaskIntegrationManager

* Replace interface with AbstractTaskIntegrationManager

* Replace getCreatedBySomeoneElseComment with add

* Jira server - store issueId and repositoryId

* Use interface, store context and info in state

* Merge Jira Server integration manager into JiraServerRestManager

* Rename GitHub integration manager to GitHubServerManager

* Make jira integration manager similar to others

* Update JiraServer icon

* add types

* Remove redundant headers

* Remove AbstractTaskIntegrationManager

* PokerEstimateHeaderCardContent

* Revert ImportedTask

Co-authored-by: Matt Krick <[email protected]>
Co-authored-by: Georg Bremer <[email protected]>
Co-authored-by: Ben Limmer <[email protected]>
jmtaber129 pushed a commit that referenced this pull request Apr 5, 2022
* Add generic createTaskIntegration mutation

* Simplify

* Remove old mutations

* Update demo

* Use existing integration provider service

* authLoaderKey instead of returning data loader

* Move db actions outside

* getCreatedBySomeoneElseComment

* Remove internal state

* Add manager factory

* Tweak variable names

* Add mask false

* Allow to use any custom auth loader

* Simplify

* fix @relay(mask: false) bug

* Change CreateTaskResponse

* Change CreateTaskResponse

* Rename projectId to integrationRepoId

* Formatting

* Add Jira Server to RepoIntegrations

* Remove added id from GitHub schema

It already has an id.

* Tweak graphql.ts

* Add JiraServerTaskintegrationManager

* addComment

* Fix comment

* Jira Server: show integrated task

* Tweak import paths for circleci build

* Add avatar fetching for Jira Server

* build(server): fix jest transforms

* Fix TypeError: jiraProjects is not iterable

* Tweak type

* Tweak isTypeOf

* use useFragment instead of createFragmentContainer

* Cleanup JiraServerIntegraiton GraphQL types

* Convert fetchJiraServerProjects into a dataloader

* Improve JiraServerRestManager error reporting

* feat(jira-server): List issues in sprint poker

* Fix JiraServerRestManager request

* Jira Server: show integrated task  (#6092)

* Jira Server: show integrated task

* Tweak import paths for circleci build

* build(server): fix jest transforms

* Tweak isTypeOf

* Tweak naming

* Do not call factory from dataloader

Co-authored-by: Ben Limmer <[email protected]>

* Fix IntegrationRepoId

* Clean up

* Fix issueKey

* Fix provider id type

* add jiraServer feature flag

* Remove jiraServer feature flag

* getApiManager

* Fix initManager type

* Add MockScopingList

* ids split - assume the input is valid and return non-null types

* Simplify

* Specific type

* Tweak poker tabs

* Call integration manager directly

* Remove fetch jira server projects

* Replace interface with AbstractTaskIntegrationManager

* Replace interface with AbstractTaskIntegrationManager

* Replace getCreatedBySomeoneElseComment with add

* Jira server - store issueId and repositoryId

* Use interface, store context and info in state

* Merge Jira Server integration manager into JiraServerRestManager

* Rename GitHub integration manager to GitHubServerManager

* Make jira integration manager similar to others

* Update JiraServer icon

* non-null addCreatedBySomeoneElseComment

Co-authored-by: Matt Krick <[email protected]>
Co-authored-by: Georg Bremer <[email protected]>
Co-authored-by: Ben Limmer <[email protected]>
jmtaber129 pushed a commit that referenced this pull request Apr 5, 2022
* Add generic createTaskIntegration mutation

* Simplify

* Remove old mutations

* Update demo

* Use existing integration provider service

* authLoaderKey instead of returning data loader

* Move db actions outside

* getCreatedBySomeoneElseComment

* Remove internal state

* Add manager factory

* Tweak variable names

* Add mask false

* Allow to use any custom auth loader

* Simplify

* fix @relay(mask: false) bug

* Change CreateTaskResponse

* Change CreateTaskResponse

* Rename projectId to integrationRepoId

* Formatting

* Add Jira Server to RepoIntegrations

* Remove added id from GitHub schema

It already has an id.

* Tweak graphql.ts

* Add JiraServerTaskintegrationManager

* addComment

* Fix comment

* Jira Server: show integrated task

* Tweak import paths for circleci build

* Add avatar fetching for Jira Server

* build(server): fix jest transforms

* Fix TypeError: jiraProjects is not iterable

* Tweak type

* Tweak isTypeOf

* use useFragment instead of createFragmentContainer

* Cleanup JiraServerIntegraiton GraphQL types

* Convert fetchJiraServerProjects into a dataloader

* Improve JiraServerRestManager error reporting

* feat(jira-server): List issues in sprint poker

* Fix JiraServerRestManager request

* Jira Server: show integrated task  (#6092)

* Jira Server: show integrated task

* Tweak import paths for circleci build

* build(server): fix jest transforms

* Tweak isTypeOf

* Tweak naming

* Do not call factory from dataloader

Co-authored-by: Ben Limmer <[email protected]>

* Fix IntegrationRepoId

* Clean up

* Fix issueKey

* Fix provider id type

* add jiraServer feature flag

* Remove jiraServer feature flag

* getApiManager

* Fix initManager type

* Add MockScopingList

* Allow adding a Jira Server task to poker scope

* Show Jira Server description

* ids split - assume the input is valid and return non-null types

* Simplify

* Specific type

* Tweak poker tabs

* Call integration manager directly

* Remove fetch jira server projects

* Replace interface with AbstractTaskIntegrationManager

* Replace interface with AbstractTaskIntegrationManager

* Replace getCreatedBySomeoneElseComment with add

* Jira server - store issueId and repositoryId

* Use interface, store context and info in state

* Merge Jira Server integration manager into JiraServerRestManager

* Rename GitHub integration manager to GitHubServerManager

* Make jira integration manager similar to others

* Update JiraServer icon

* add types

* Remove redundant headers

* Remove AbstractTaskIntegrationManager

* PokerEstimateHeaderCardContent

* Revert ImportedTask

Co-authored-by: Matt Krick <[email protected]>
Co-authored-by: Georg Bremer <[email protected]>
Co-authored-by: Ben Limmer <[email protected]>
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.

Jira Server: show integrated task
3 participants