Skip to content
This repository has been archived by the owner on Oct 23, 2023. It is now read-only.

Project level attributes via matchable resource #320

Merged
merged 4 commits into from
Sep 29, 2022
Merged

Conversation

wild-endeavor
Copy link
Contributor

@wild-endeavor wild-endeavor commented Sep 13, 2022

TL;DR

This IDL change implements a partial solution to the broader settings project. The main purpose of this change is to get this section of the project domain view to show up all the time.

image

Having project-level matchable resources also makes sense in the long run. This PR adds

  • GetProjectAttributes
  • UpdateProjectAttributes
  • DeleteProjectAttributes

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

This is a looser (lower specificity) version of the project-domain settings.

Tracking Issue

See comment above.

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Sep 13, 2022

Codecov Report

Merging #320 (cc8779f) into master (c4ea1f9) will not change coverage.
The diff coverage is n/a.

❗ Current head cc8779f differs from pull request most recent head f271ff3. Consider uploading reports for the commit f271ff3 to get more accurate results

@@           Coverage Diff           @@
##           master     #320   +/-   ##
=======================================
  Coverage   75.04%   75.04%           
=======================================
  Files          18       18           
  Lines        1162     1162           
=======================================
  Hits          872      872           
  Misses        237      237           
  Partials       53       53           
Flag Coverage Δ
unittests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>
@wild-endeavor wild-endeavor changed the title [wip] project level attributes Project level attributes Sep 27, 2022
Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>
@wild-endeavor wild-endeavor changed the title Project level attributes Project level attributes via matchable resource Sep 27, 2022
Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>
@wild-endeavor wild-endeavor marked this pull request as ready for review September 27, 2022 21:28
@EngHabu EngHabu merged commit 245e645 into master Sep 29, 2022
@EngHabu EngHabu deleted the project-level branch September 29, 2022 17:39
eapolinario pushed a commit that referenced this pull request Sep 8, 2023
* copy pasta

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>

* generate

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>

* fix comment

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants