Draft: Add environment object page
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.
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?
-
The MR title and commit message meet the Pajamas commit conventions -
Add a description to this MR explaining the reasons for the proposed change, per say-why-not-just-what -
Include MR scope that reflects the purpose of the change in the description - For example, if this MR is focused on
usage guidelines
, addressingaccessibility challenges
could be added separately with a different MR
- For example, if this MR is focused on
-
If creating a new page on Pajamas from scratch, follow the page template structure -
Follow the Pajamas Voice & Tone Guidelines and GitLab Documentation Style Guide -
Link components to and from other related components and patterns on the .md
file, where helpful -
If embedding a Figma file, follow the Figma embed guide -
Assign it to a reviewer. All GitLab designers are reviewers, but you can also assign it directly to a maintainer or trainee maintainer -
Apply the correct component label(s) -
Select a milestone or due date
Links
Merge request reports
Activity
added Pajamas label
assigned to @dfosco
@pedroms @npost Here's the Environment conceptual model I've been working on, based on your work for the MR object page
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:
- As someone working outside of the Ops section, is this content useful and understandable for you?
- 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
? - 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!
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
@dfosco thanks for working on this and taking the time to contribute it! I'm so excited to see others thinking about objects
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
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
@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
@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
added 1 commit
- 56aff19d - docs(objects) Add newline at the end of environment object page
1 Error 253eb5eb: The commit does not comply with conventional commits specifications. 1 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
DangerEdited by 🤖 GitLab Bot 🤖mentioned in issue gitlab-org/gitlab#213589 (closed)
requested review from @tauriedavis
requested review from @pedroms
@pedroms Do you mind reviewing this first since you are familiar with objects?
removed review request for @tauriedavis
- contents/objects/environment.md 0 → 100644
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
nitpick (non-blocking): small grammar fix and putting the link on the same line5 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/)
- contents/objects/environment.md 0 → 100644
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
nitpick (non-blocking): lowercase environment19 <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">
- contents/objects/environment.md 0 → 100644
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] @dfosco
nitpick (non-blocking): consider clarifying the acronym
- contents/objects/environment.md 0 → 100644
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
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!
- contents/objects/environment.md 0 → 100644
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
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.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)).
@dfosco
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 withCloses {issue-url-or-id}
to the MR description.@dfosco
chore: please add this new page to the navigation in https://gitlab.com/gitlab-org/gitlab-services/design.gitlab.com/-/blob/main/nav.json#L295-297removed 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
Toggle commit list-
931057dc...7741b538 - 32 commits from branch
mentioned in issue gitlab-org/ci-cd-ux#48 (closed)
mentioned in issue gitlab-org/ci-cd-ux#49 (closed)
added environments label
added pajamasdefine label
mentioned in issue dfosco/notes#21 (closed)
mentioned in issue gitlab-org/ci-cd/release-group/release#105 (closed)
changed milestone to %14.9
unassigned @dfosco
assigned to @dfosco
mentioned in issue gitlab-org/gitlab-design#1824 (closed)
- Resolved by Rayana Verissimo
@pedroms how do you feel about this MR? Should we close it?
requested review from @pedroms
removed review request for @pedroms
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
or a 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!