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

Team/Org discussions: Initial commit #14644

Closed
wants to merge 10 commits into from
Closed

Conversation

Kreyren
Copy link
Contributor

@Kreyren Kreyren commented Feb 11, 2021

DO NOT MERGE

This merge request is work in progress published here as draft to coordinate the work with the rest of the team.


Pair-programming: https://go.codetogether.com/#/051cb379-92f2-40e8-8c83-86e7c00441d2/GHkQneZm1ahbShpHhI6Jpu

Fixes: #14562

Current status

Rework of the merge request

TODO

  • Rework the merge request based on new finding and brainstorms.
  • Initial frontend implemented
    • Page with button "Ping"
  • Initial backend implemented
    • Button "Ping" outputs "PING" in console
    • Button "Submit" outputs content of message field above it in console
  • Data management
    • TBD
  • Full Backend implementation
    • TBD
  • Full Frontend implemented
    • Convert the current issue form in a template usable for discussions
    • Propose mock-up to upstream
    • TBD
  • API implementation
    • TBD
  • Make discussions opt-in
    • Per Organization
    • Per Team
  • Resolve DNM (Do Not Merge) tags
  • Optimize CI/CD
  • Update documentation
  • Handle legality
  • Implement translations
  • Remove Docker Refactor used here only for testing (proposed in Docker: Rework #14626)
  • Provide summary of a design and decisions in merge request's original post

Brainstorm: Data management

TBD

Signed-off-by: Jacob Hrbek [email protected]


Merge process

Squash commits using GitHub UI with topic and message from original post of a merge request without 'Merge Process'

@Kreyren Kreyren changed the title Team discussions: Initial commit Team/Org discussions: Initial commit Feb 18, 2021
@Kreyren
Copy link
Contributor Author

Kreyren commented Mar 13, 2021

Transferred in Kreytea#1 will be submitted to upstream once finished.

@Kreyren Kreyren closed this Mar 13, 2021
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 13, 2021
@go-gitea go-gitea locked and limited conversation to collaborators May 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Team discussions in gitea?
2 participants