[ENG-9111] Reverted locking access_requests_enabled behind api version (#11349)

* reverted locking access_requests_enabled behind api version in NodeSerializer
* fixed test for access_request

Author: opaduchak

api/nodes/serializers.py

@@ -272,7 +272,7 @@ class NodeSerializer(TaxonomizableSerializerMixin, JSONAPISerializer):
     fork = ser.BooleanField(read_only=True, source='is_fork')
     collection = ser.BooleanField(read_only=True, source='is_collection')
     tags = ValuesListField(attr_name='name', child=ser.CharField(), required=False)
-    access_requests_enabled = ShowIfVersion(ser.BooleanField(read_only=False, required=False), min_version='2.0', max_version='2.8')
+    access_requests_enabled = ser.BooleanField(read_only=True, required=False)
     node_license = NodeLicenseSerializer(required=False, source='license')
     # 'analytics_key' can be removed after another version release.
     analytics_key = ShowIfAdminScopeOrAnonymous(ser.CharField(read_only=True, default=''))

api_tests/nodes/views/test_node_detail.py

@@ -358,11 +358,9 @@ def test_preprint_field(self, app, user, user_two, project_public, url_public):
         assert res.status_code == 200
         assert res.json['data']['attributes']['preprint'] is False
 
-    def test_shows_access_requests_enabled_field_based_on_version(self, app, user, project_public, url_public):
+    def test_shows_access_requests_enabled_field(self, app, user, project_public, url_public):
         url = url_public + '?version=latest'
         res = app.get(url, auth=user.auth)
-        assert 'access_requests_enabled' not in res.json['data']['attributes']
-        res = app.get(url_public + '?version=2.8', auth=user.auth)
         assert 'access_requests_enabled' in res.json['data']['attributes']
 
     def test_node_shows_correct_templated_from_count(self, app, user, project_public, url_public):