Merge remote-tracking branch 'origin/master' into repp-domains

CHANGELOG.md

@@ -1,3 +1,11 @@
+06.01.2021
+* Reverted Nokogiri bump due to dependency conflicts in production [#1787](https://github.com/internetee/registry/pull/1787)
+
+05.01.2021
+* Fixed ok/inactive bug together with disclosed contact attribute handling [#1786](https://github.com/internetee/registry/pull/1786)
+* Ident data to simplified domain list in registrant API to help with sorting and filtering [#1783](https://github.com/internetee/registry/pull/1783)
+* Bumped Nokogiri to 1.11.0 [#1785](https://github.com/internetee/registry/pull/1785)
+
 23.12.2020
 * fix for REPP logging and registrar portal communication [#1782](https://github.com/internetee/registry/pull/1782)
 

app/models/domain.rb

@@ -574,7 +574,7 @@ class Domain < ApplicationRecord
 
     if statuses.empty? && valid?
       statuses << DomainStatus::OK
-    elsif (statuses.length > 1 && active?) || !valid?
+    elsif (statuses.length > 1) || !valid?
       statuses.delete(DomainStatus::OK)
     end
 

app/models/epp/domain.rb

@@ -29,7 +29,7 @@ class Epp::Domain < Domain
     active_techs = tech_domain_contacts.select { |x| !x.marked_for_destruction? }
 
     # validate registrant here as well
-    ([registrant] + active_admins + active_techs).each do |x|
+    ([Contact.find_by(code: registrant.code)] + active_admins + active_techs).each do |x|
       unless x.valid?
         add_epp_error('2304', nil, nil, I18n.t(:contact_is_not_valid, value: x.code))
         ok = false

lib/serializers/registrant_api/domain.rb

@@ -59,7 +59,8 @@ module Serializers
           registrant_verification_asked_at: domain.registrant_verification_asked_at,
           registrar: { name: domain.registrar.name, website: domain.registrar.website },
           registrant: { name: domain.registrant.name, id: domain.registrant.uuid,
-                        phone: domain.registrant.phone, email: domain.registrant.email }
+                        phone: domain.registrant.phone, email: domain.registrant.email,
+                        ident: domain.registrant.ident }
         }
       end