Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

feat(decouple): openy_node_alert #2484

Merged
merged 2 commits into from
Jun 29, 2021
Merged

Conversation

podarok
Copy link
Contributor

@podarok podarok commented Jun 25, 2021

Testing decouple is good before merging open-y-subprojects/openy_features#2 and releasing new version of https://github.com/open-y-subprojects/openy_features/releases

Original Issue, this PR is going to fix: https://openy.atlassian.net/browse/MAINTAIN-93

Make sure these boxes are checked before asking for review of your pull request - thank you!

If there is a new feature or this is a bug fix - use 9.x-2.x branch. We'll tag for release if the bug is critical asap or tag for release next bug fix release until critical issue arrived.

Steps for review

General checks

Thank you for your contribution!

@podarok podarok added PR: DO NOT MERGE Work in Progress or something not to be merged at all Level: Middle Needs skilled developer to work on Type: Task Maintenance, subtasks Decoupling labels Jun 25, 2021
@podarok podarok self-assigned this Jun 25, 2021
@podarok podarok changed the title feat(decouple) feat(decouple): openy_node_alert Jun 25, 2021
@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3240/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3240/

@podarok podarok added this to the Decouple first initiative milestone Jun 25, 2021
@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3240/

@podarok podarok added PR: Needs Review Needs someone review ( code ) PR: Needs Testing Manual testing is needed PR: Needs Work Unfinished task. Issues still there labels Jun 25, 2021
@podarok podarok removed PR: DO NOT MERGE Work in Progress or something not to be merged at all PR: Needs Work Unfinished task. Issues still there labels Jun 25, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 25, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Jun 25, 2021
@podarok
Copy link
Contributor Author

podarok commented Jun 25, 2021

retest this please

@gundevel
Copy link
Collaborator

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3250/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3250/

@gundevel
Copy link
Collaborator

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3250/

@podarok
Copy link
Contributor Author

podarok commented Jun 25, 2021

image

@podarok podarok added PR: Tested Manually tested. Green light for merging and removed PR: Needs Testing Manual testing is needed PR: Needs Review Needs someone review ( code ) labels Jun 29, 2021
@podarok podarok added the PR: Code Reviewed Code reviewed. Ready for QA label Jun 29, 2021
@podarok podarok merged commit f3262d6 into 9.x-2.x Jun 29, 2021
@podarok podarok deleted the 9.x-2.x-openy_features_alerts branch June 29, 2021 11:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Decoupling Level: Middle Needs skilled developer to work on PR: Code Reviewed Code reviewed. Ready for QA PR: Tested Manually tested. Green light for merging Type: Task Maintenance, subtasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants