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

Remove Fullname search parameter from status board product, application, and service. #875

Merged
merged 5 commits into from
Feb 8, 2024

Conversation

tinaafitz
Copy link
Contributor

ocm-api-model changes: openshift-online/ocm-api-model#872.

@tinaafitz
Copy link
Contributor Author

Hi @smcavey, Please review. :-)

@smcavey
Copy link

smcavey commented Jan 31, 2024

Is there a model version change that should be associated with these changes?

@tinaafitz
Copy link
Contributor Author

Hi @smcavey

Is there a model version change that should be associated with these changes?

Yes. I added a commit to update the version. :-)

@smcavey
Copy link

smcavey commented Feb 2, 2024

lgtm :)

@tinaafitz tinaafitz force-pushed the remove_fullname branch 2 times, most recently from a238274 to 849fd2c Compare February 5, 2024 16:32
@tinaafitz
Copy link
Contributor Author

Hi @tzvatot, Can you review this MR when you have time? Thank you so much. :-)

CHANGES.md Outdated Show resolved Hide resolved
CHANGES.md Outdated
## 0.1.399
- Remove `StatusBoard` `fullname` search parameter.
- Deprecate `Notify` resource.
- Update model version v0.0.352.
Copy link
Contributor

Choose a reason for hiding this comment

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

CHANGES.md Outdated Show resolved Hide resolved
Copy link
Contributor

@tzvatot tzvatot left a comment

Choose a reason for hiding this comment

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

LGTM

@tzvatot tzvatot merged commit 35483fd into openshift-online:main Feb 8, 2024
3 checks passed
@tzvatot
Copy link
Contributor

tzvatot commented Feb 8, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants