Skip to content
Snippets Groups Projects

Draft: Add environment object page

Closed Daniel Fosco requested to merge add-environment-object into main
6 unresolved threads

What does this MR do?

Adds an object documentation page for Environment with object description and conceptual model (see screenshot below)

Still Draft, as the layout mapping section is still ongoing.

image

Figma Link in page: https://www.figma.com/file/0pQXky3SCtdzydXRZSQZm8/Release-Group-%E2%80%A2-Ops-Object-Models?node-id=1%3A740

Does this MR meet the acceptance criteria?

Links

Edited by Daniel Fosco

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added Pajamas label

  • assigned to @dfosco

  • Daniel Fosco changed the description

    changed the description

    • @pedroms @npost Here's the Environment conceptual model I've been working on, based on your work for the MR object page :slight_smile:

      This has been useful in helping me understand the scope and boundaries of Environments, and will serve as the base of the upcoming UI work on the page -- I'm currently working on the Layout section of the doc.

      Questions:

      1. As someone working outside of the Ops section, is this content useful and understandable for you?
      2. The conceptual model description seems relevant to any object created, perhaps it's better to add it in a separate page from Environment and MR, such as How to add a new object?
      3. I plan on adding other objects later (Deployment, Release, etc), but does it makes sense to have the entire relationship map represented somewhere?

      Let me know if you have any other feedbacks!

    • Contributor

      Thanks for the ping @dfosco

      It would be very useful before answering these questions to see a quick video overview of you telling the story of this object and how it fits into the broader system.

    • @npost Great idea! Will put a short video together and share it here :thumbsup:

    • @dfosco thanks for working on this and taking the time to contribute it! I'm so excited to see others thinking about objects :slight_smile: Sorry for the delay reviewing this, I intend to get to it during this week.

      In the meantime, I think Nick's suggestion to create a short walkthrough video wouldn't hurt, if you have the time. Thanks in advance :bow:

    • I would actually caution against the walkthrough video. We should be able to read the docs and understand them based on that. Its a good exercise to go into the documentation blind and see what, if anything, needs to be updated in order to fully understand the content.

      After reading the content, and if it is not well understood, then a walkthrough video could be helpful in order to determine what needs to be added. But I would suggest writing down the questions you have and then having Daniel update the MR to address those questions.

      Edited by Taurie Davis
    • @tauriedavis That's a good point! I can definitely expand the description and add better examples, environments can be pretty abstract sometimes.

      @pedroms @npost Let me know if you have specific questions, but I'll start expanding the content and add to this MR regardless :slight_smile:

    • @tauriedavis that's fair. As I wrote, a video was not a requirement, I was still going to review this without it.

      @dfosco I'd prefer to review the content of this MR independently from the conceptual model you are working on in Figma. For that, would you mind creating a branch from the main Conceptual model Figma for your work on Environments? Let me know when that's done so I can review it in place :open_hands:

    • Please register or sign in to reply
  • @tauriedavis Let me know if there's anything I should be doing different in the format for this MR, it's my first doc contribution to Pajamas

  • Daniel Fosco marked the checklist item If [embedding a Figma file][figma-embed], follow the [Figma embed guide][figma-embed-guide] as completed

    marked the checklist item If [embedding a Figma file][figma-embed], follow the [Figma embed guide][figma-embed-guide] as completed

  • Daniel Fosco added 1 commit

    added 1 commit

    • 56aff19d - docs(objects) Add newline at the end of environment object page

    Compare with previous version

  • 1 Error
    :no_entry_sign: 253eb5eb: The commit does not comply with conventional commits specifications.
    1 Warning
    :warning: This MR will trigger no release

    Commit message standards

    One or more commit messages do not meet our Git commit message standards. For more information on how to write a good commit message, take a look at Conventional commits.

    Here is an example of a good commit message:

    feat(progressbar): Improve rendering performance in Edge
    
    Our progressbar component was causing a lot of re-renders in Edge.
    This was caused by excessive re-rendering due to floating point errors
    in Edge's JavaScript engine.
    
    By utilizing the better-math library we avoid those calculation errors
    and we can achieve 120 rendering frames per second again.

    This is an example of a bad commit message:

    fixed progressbar

    No release

    This Merge Request will trigger no release. This either means, no commit neither the MR title warrant a semantic release (e.g. only updating CI config); or that all commits and/or the MR title are not properly formatted according to conventional commits rules, in the latter case, you should either:

    • Disable Squash commits when merge request is accepted and rewrite the commits history to format commit messages properly
    • Enable Squash commits when merge request is accepted and make sure the MR's title complies with conventional commits specifications

    Generated by :no_entry_sign: Danger

    Edited by 🤖 GitLab Bot 🤖
  • Daniel Fosco added 2 commits

    added 2 commits

    • dfe7e30b - docs(objects): Add environment object page
    • e063d59d - docs(objects): Add newline at the end of environment object page

    Compare with previous version

  • Daniel Fosco added 1 commit

    added 1 commit

    • 931057dc - docs(objects): Update Figma embed URL

    Compare with previous version

  • Taurie Davis requested review from @tauriedavis

    requested review from @tauriedavis

  • Taurie Davis requested review from @pedroms

    requested review from @pedroms

  • @pedroms Do you mind reviewing this first since you are familiar with objects?

  • Taurie Davis removed review request for @tauriedavis

    removed review request for @tauriedavis

1 ---
2 name: Environment
3 ---
4
5 An Environment describe where code is deployed. In more technical terms, an environment is a deployment target defined by the user.
6
7 [How are Environments used in the product?](https://docs.gitlab.com/ee/ci/environments/)
  • Comment on lines +5 to +7

    @dfosco :pick: nitpick (non-blocking): small grammar fix and putting the link on the same line

    Suggested change
    5 An Environment describe where code is deployed. In more technical terms, an environment is a deployment target defined by the user.
    6
    7 [How are Environments used in the product?](https://docs.gitlab.com/ee/ci/environments/)
    5 An Environment describes where code is deployed. In more technical terms, an environment is a deployment target defined by the user. [How are environments used in the product?](https://docs.gitlab.com/ee/ci/environments/)
  • Please register or sign in to reply
  • 4
    5 An Environment describe where code is deployed. In more technical terms, an environment is a deployment target defined by the user.
    6
    7 [How are Environments used in the product?](https://docs.gitlab.com/ee/ci/environments/)
    8
    9 ## Conceptual model
    10
    11 This model allows for proper separation of concerns: concepts (meaning) and user interface (representation; see [layouts](#layouts)).
    12 In other words, a conceptual model is only focused on the concepts and how they relate to each other, not how it’s technically implemented or used in the UI.
    13
    14 A conceptual model defines the **objects** and **actions**, and enumerates the **attributes** for every object.
    15 The language of the conceptual model diagram is meant to be as simple and accessible as possible to everyone, even if they are not familiar with these kinds of diagrams (for more information, see [Object Modeling for Designers](https://medium.com/@hpadkisson/object-modeling-for-designers-an-introduction-7871bdcf8baf).
    16
    17 [View conceptual model in Figma →](https://www.figma.com/file/0pQXky3SCtdzydXRZSQZm8/Release-Group-%E2%80%A2-Ops-Object-Models?node-id=1%3A786)
    18
    19 <div class="figma-embed" aria-label="Conceptual diagram connecting objects, along with their attributes and actions, to the primary Environment object." role="img">
    • @dfosco :pick: nitpick (non-blocking): lowercase environment

      Suggested change
      19 <div class="figma-embed" aria-label="Conceptual diagram connecting objects, along with their attributes and actions, to the primary Environment object." role="img">
      19 <div class="figma-embed" aria-label="Conceptual diagram connecting objects, along with their attributes and actions, to the primary environment object." role="img">
    • Please register or sign in to reply
  • 10
    11 This model allows for proper separation of concerns: concepts (meaning) and user interface (representation; see [layouts](#layouts)).
    12 In other words, a conceptual model is only focused on the concepts and how they relate to each other, not how it’s technically implemented or used in the UI.
    13
    14 A conceptual model defines the **objects** and **actions**, and enumerates the **attributes** for every object.
    15 The language of the conceptual model diagram is meant to be as simple and accessible as possible to everyone, even if they are not familiar with these kinds of diagrams (for more information, see [Object Modeling for Designers](https://medium.com/@hpadkisson/object-modeling-for-designers-an-introduction-7871bdcf8baf).
    16
    17 [View conceptual model in Figma →](https://www.figma.com/file/0pQXky3SCtdzydXRZSQZm8/Release-Group-%E2%80%A2-Ops-Object-Models?node-id=1%3A786)
    18
    19 <div class="figma-embed" aria-label="Conceptual diagram connecting objects, along with their attributes and actions, to the primary Environment object." role="img">
    20 <iframe frameborder="0" src="https://www.figma.com/embed?embed_host=share&url=https%3A%2F%2Fwww.figma.com%2Ffile%2F0pQXky3SCtdzydXRZSQZm8%2FRelease-Group-%25E2%2580%25A2-Ops-Object-Models%3Fnode-id%3D1%253A786" allowfullscreen></iframe>
    21 </div>
    22
    23 ## Layouts
    24
    25 [WIP]
  • 1 ---
    2 name: Environment
    3 ---
    4
    5 An Environment describe where code is deployed. In more technical terms, an environment is a deployment target defined by the user.
    6
    7 [How are Environments used in the product?](https://docs.gitlab.com/ee/ci/environments/)
    8
    9 ## Conceptual model
    10
    11 This model allows for proper separation of concerns: concepts (meaning) and user interface (representation; see [layouts](#layouts)).
    12 In other words, a conceptual model is only focused on the concepts and how they relate to each other, not how it’s technically implemented or used in the UI.
    13
    14 A conceptual model defines the **objects** and **actions**, and enumerates the **attributes** for every object.
    15 The language of the conceptual model diagram is meant to be as simple and accessible as possible to everyone, even if they are not familiar with these kinds of diagrams (for more information, see [Object Modeling for Designers](https://medium.com/@hpadkisson/object-modeling-for-designers-an-introduction-7871bdcf8baf).
    • Comment on lines +11 to +15

      @dfosco :bulb: thought (non-blocking): this introductory text will be applicable to all objects that are added to Pajamas, so it might make sense in the future to create an "Overview" page for the “Objects” section with this content, to avoid duplication and keep things DRY. Just a thought, non-blocking! :slight_smile:

    • Please register or sign in to reply
  • 1 ---
    2 name: Environment
    3 ---
    4
    5 An Environment describe where code is deployed. In more technical terms, an environment is a deployment target defined by the user.
    6
    7 [How are Environments used in the product?](https://docs.gitlab.com/ee/ci/environments/)
    8
    9 ## Conceptual model
    10
    11 This model allows for proper separation of concerns: concepts (meaning) and user interface (representation; see [layouts](#layouts)).
    12 In other words, a conceptual model is only focused on the concepts and how they relate to each other, not how it’s technically implemented or used in the UI.
    13
    14 A conceptual model defines the **objects** and **actions**, and enumerates the **attributes** for every object.
    15 The language of the conceptual model diagram is meant to be as simple and accessible as possible to everyone, even if they are not familiar with these kinds of diagrams (for more information, see [Object Modeling for Designers](https://medium.com/@hpadkisson/object-modeling-for-designers-an-introduction-7871bdcf8baf).
    • @dfosco :pick: nitpick: missing ending parentheses. This is also missing in the MR object page — I'm going to add a commit to this MR to fix it, if you don't mind.

      Suggested change
      15 The language of the conceptual model diagram is meant to be as simple and accessible as possible to everyone, even if they are not familiar with these kinds of diagrams (for more information, see [Object Modeling for Designers](https://medium.com/@hpadkisson/object-modeling-for-designers-an-introduction-7871bdcf8baf).
      15 The language of the conceptual model diagram is meant to be as simple and accessible as possible to everyone, even if they are not familiar with these kinds of diagrams (for more information, see [Object Modeling for Designers](https://medium.com/@hpadkisson/object-modeling-for-designers-an-introduction-7871bdcf8baf)).
    • Please register or sign in to reply
  • @dfosco :wrench: chore (non-blocking): Is there an issue we can link to in the MR description? Maybe there's an issue that would be closed by this MR? If so, you can add it with Closes {issue-url-or-id} to the MR description.

  • Pedro Moreira da Silva removed review request for @pedroms

    removed review request for @pedroms

  • added 35 commits

    • 931057dc...7741b538 - 32 commits from branch main
    • 504c3a16 - docs(objects): Add environment object page
    • 3076583a - docs(objects): Add newline at the end of environment object page
    • 03ed41d1 - docs(objects): Update Figma embed URL

    Compare with previous version

  • added 1 commit

    • 253eb5eb - Add ending parentheses in MR object page

    Compare with previous version

  • Daniel Fosco changed milestone to %14.9

    changed milestone to %14.9

  • unassigned @dfosco

  • assigned to @dfosco

  • Daniel Fosco assigned to @rayana and unassigned @dfosco

    assigned to @rayana and unassigned @dfosco

  • Rayana Verissimo requested review from @pedroms

    requested review from @pedroms

  • Pedro Moreira da Silva removed review request for @pedroms

    removed review request for @pedroms

  • Rayana Verissimo mentioned in issue #1257

    mentioned in issue #1257

  • Hi @dfosco,

    We would love to know how you found your code review experience in this merge request! Please leave a :thumbsup: or a :thumbsdown: on this comment to describe your experience.

    Once done, please comment @gitlab-bot feedback below and feel free to leave any additional feedback you have in the same comment.

    You can also fill out a 5 minute survey to provided additional feedback on how GitLab can improve the contributor experience.

    Thanks for your help! :heart:

  • Please register or sign in to reply
    Loading