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

crosslink - links from system and activation key subscriptions #400

Merged
merged 1 commit into from
Aug 1, 2012

Conversation

thomasmckay
Copy link
Member

Depending on whether the subscription is from Red Hat provider or a custom one, links are now shown on Activation Keys and Systems for the displayed subscriptions.

@ghost ghost assigned iNecas Jul 31, 2012
@iNecas
Copy link
Member

iNecas commented Jul 31, 2012

I was not able to subscribe a system with activation key asssigned to a subscription: getting

Pool ff80808138dc0e040138dc19d3f30032 has no product associated

This is not probably caused byt this pull request, but rather with this one c80fac9 and the following code

remote_values.each_pair {|k,v| instance_variable_set("@#{k.to_s}", v) if (attrs and attrs.include?(k.to_sym) and respond_to?("#{k.to_s}="))}
, expecting the variables and reponse keys to have the same name.

iNecas added a commit that referenced this pull request Aug 1, 2012
crosslink - links from system and activation key subscriptions
@xsuchy xsuchy merged commit 7933b04 into Katello:master Aug 1, 2012
@iNecas
Copy link
Member

iNecas commented Aug 1, 2012

My previous comment doesn't relate directly to this pool request. The code changes here are ok, I will send fiexes for the found issues in different pullrequest. Merging now.

iNecas added a commit that referenced this pull request Apr 27, 2013
crosslink - links from system and activation key subscriptions
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