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

Changed MenuTree Names (bsc#1019847) #259

Merged
merged 3 commits into from
Nov 7, 2018
Merged

Changed MenuTree Names (bsc#1019847) #259

merged 3 commits into from
Nov 7, 2018

Conversation

amannch
Copy link

@amannch amannch commented Oct 30, 2018

What does this PR change?

Updated localization Strings for the Menu on the
left hand side of the web UI.
This is an attempt to remove redundant
names so that it is easier to navigate
the site for the user.

This PR is ported from https://github.com/SUSE/spacewalk/pull/5929

GUI diff

See https://github.com/SUSE/spacewalk/pull/5929

  • DONE

Documentation

#271

  • DONE

Test coverage

No Errors on local testsuite for 3.1
Uyuni-testsuite couldn't be run due to postgres errors when creating an activation key.
(Seems to be unrelated to the changes in this PR)

  • DONE

https://github.com/SUSE/spacewalk/issues/5854
https://bugzilla.suse.com/show_bug.cgi?id=1019847

Relevant branches (GitHub automatic links expected below):

  • Uyuni

  • DONE

@amannch amannch changed the title Master bsc1019847 Changed MenuTree Names (bsc#1019847) Oct 30, 2018
Copy link
Contributor

@ncounter ncounter left a comment

Choose a reason for hiding this comment

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

It would be nice if you can give a testsuite run on a Uyuni setup just to be 100% sure we will not break it.
I don't expect any diff actually so I'm early approving the PR.

Thank you 👍

@amannch
Copy link
Author

amannch commented Oct 30, 2018

It would be nice if you can give a testsuite run on a Uyuni setup just to be 100% sure we will not break it.

Yes, I won't merge until I got a successful testrun - thanks for the quick review

Copy link
Member

@juliogonzalez juliogonzalez left a comment

Choose a reason for hiding this comment

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

LGTM. Pre-approved with same comment as @ncounter :-)

@amannch
Copy link
Author

amannch commented Nov 7, 2018

After some adjustments for uyuni the UI-changes work fine with the testsuite.
So what comes next: merging or telling the documentation guys about the changes?

@ncounter
Copy link
Contributor

ncounter commented Nov 7, 2018

@amannch I created an issue for documentation update, and this is enough for "telling the documentation guys about the changes". Feel free to edit the issue as you wish. Then, just merge it 😉

Christian Amann added 3 commits November 7, 2018 12:54
Updated localization Strings for the Menu on the
left hand side of the web UI.
This is an attempt to remove redundant and/or
duplicate names so that it is easier to navigate
the site for the user
Changes entry names to run with changes made
in commit: 9720f2261ae3c9300ed117d38c5ee2385fae8762
@amannch amannch merged commit 35a0958 into master Nov 7, 2018
@amannch amannch deleted the master-bsc1019847 branch November 7, 2018 12:12
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.

None yet

3 participants