Skip to content

feat(bulk-import): add fields for annotations, labels and spec input #1950

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

Conversation

ciiay
Copy link
Contributor

@ciiay ciiay commented Jul 24, 2024

For RHIDP-2927

Screen recording:

rhidp_2927.mov

Copy link

openshift-ci bot commented Jul 24, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign nickboldt for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ciiay ciiay force-pushed the rhidp-2927-add-fields-for-annotations-labels-spec branch from 5a01b0a to 2bb1fd1 Compare July 24, 2024 19:08
@ciiay ciiay marked this pull request as ready for review July 24, 2024 19:08
@ciiay ciiay requested a review from a team as a code owner July 24, 2024 19:08
@openshift-ci openshift-ci bot requested review from 04kash and dzemanov July 24, 2024 19:08
@ciiay ciiay changed the title feat(bulk-import): added fields for annotations, labels and spec input feat(bulk-import): add fields for annotations, labels and spec input Jul 24, 2024
@ciiay ciiay force-pushed the rhidp-2927-add-fields-for-annotations-labels-spec branch from 2bb1fd1 to 0a86cc4 Compare July 24, 2024 19:09
@ciiay ciiay requested review from debsmita1 and ShiranHi July 24, 2024 22:01
@ShiranHi
Copy link

Looks good to me, thank you!

Copy link
Member

@debsmita1 debsmita1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ciiay The newly added fields have no validation

@ciiay ciiay force-pushed the rhidp-2927-add-fields-for-annotations-labels-spec branch from d2cb2ea to 4bc6f8d Compare July 31, 2024 21:46
@ciiay ciiay requested a review from debsmita1 August 1, 2024 13:52
@ciiay ciiay force-pushed the rhidp-2927-add-fields-for-annotations-labels-spec branch 4 times, most recently from 82ed156 to cdc8f77 Compare August 1, 2024 19:35
Copy link
Member

@debsmita1 debsmita1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when you add spec/label/annotations and then remove it, they appear as empty objects in the preview section

Screenshot 2024-08-05 at 9 47 13 PM

Validation works like a charm. Thank you

@ciiay ciiay force-pushed the rhidp-2927-add-fields-for-annotations-labels-spec branch 2 times, most recently from 0d3bedf to 4da2d33 Compare August 5, 2024 17:38
@ciiay ciiay requested a review from debsmita1 August 5, 2024 17:39
@debsmita1
Copy link
Member

Tested it locally and it works fine
Please resolve the 3 issues reported by the quality gate and we will be good to go
Thanks ! :)

@ciiay ciiay force-pushed the rhidp-2927-add-fields-for-annotations-labels-spec branch from 6a0a108 to a5c1143 Compare August 5, 2024 19:34
Copy link

sonarqubecloud bot commented Aug 5, 2024

@debsmita1
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Aug 6, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit a1b790a into janus-idp:main Aug 6, 2024
9 checks passed
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## 1.0.0 (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a))
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## @janus-idp/backstage-plugin-bulk-import [1.2.0](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a))
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## [1.17.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a))
* **rbac:** nested condition ([#1814](#1814)) ([228f1a9](228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([#2009](#2009)) ([8c28891](8c28891))
* **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
janus-idp bot pushed a commit that referenced this pull request Aug 6, 2024
## [4.7.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([#1950](#1950)) ([a1b790a](a1b790a))
* **rbac:** nested condition ([#1814](#1814)) ([228f1a9](228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([#2012](#2012)) ([97c9667](97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([#2009](#2009)) ([8c28891](8c28891))
* **rbac:** implement conditional aliases ([#1847](#1847)) ([dbc9a0b](dbc9a0b))
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
## 1.0.0 (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a))
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
## @janus-idp/backstage-plugin-bulk-import [1.2.0](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a))
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
## [1.17.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a))
* **rbac:** nested condition ([janus-idp#1814](janus-idp#1814)) ([228f1a9](janus-idp@228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([janus-idp#2012](janus-idp#2012)) ([97c9667](janus-idp@97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([janus-idp#2009](janus-idp#2009)) ([8c28891](janus-idp@8c28891))
* **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
debsmita1 pushed a commit to debsmita1/backstage-plugins that referenced this pull request Aug 9, 2024
## [4.7.3](https://github.com/janus-idp/backstage-plugins/compare/@janus-idp/[email protected]...@janus-idp/[email protected]) (2024-08-06)

### Features

* **bulk-import:** add fields for annotations, labels and spec input ([janus-idp#1950](janus-idp#1950)) ([a1b790a](janus-idp@a1b790a))
* **rbac:** nested condition ([janus-idp#1814](janus-idp#1814)) ([228f1a9](janus-idp@228f1a9))

### Bug Fixes

* **aap:** revert downgrade of janus-cli ([janus-idp#2012](janus-idp#2012)) ([97c9667](janus-idp@97c9667))
* **acr:** revert downgrade of shared-react and janus-cli ([janus-idp#2009](janus-idp#2009)) ([8c28891](janus-idp@8c28891))
* **rbac:** implement conditional aliases ([janus-idp#1847](janus-idp#1847)) ([dbc9a0b](janus-idp@dbc9a0b))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants