mirror of
				https://github.com/zulip/zulip.git
				synced 2025-10-30 19:43:47 +00:00 
			
		
		
		
	register: Include web-public streams in "streams" field of response.
The "streams" field in "/register" response did not include web-public streams for non-admin users but the data for those are eventually included in the subscriptions data sent using "subscriptions", "unsubscribed" and "never_subscribed" fields. This commit adds code to include the web-public streams in "streams" field as well as everyone can access those and will make the "streams" data complete.
This commit is contained in:
		
				
					committed by
					
						 Alex Vandiver
						Alex Vandiver
					
				
			
			
				
	
			
			
			
						parent
						
							edf1de5416
						
					
				
				
					commit
					c90fd388c8
				
			| @@ -18,6 +18,14 @@ clients should check the `zulip_feature_level` field, present in the | |||||||
| /register`](/api/register-queue) responses, to determine the API | /register`](/api/register-queue) responses, to determine the API | ||||||
| format used by the Zulip server that they are interacting with. | format used by the Zulip server that they are interacting with. | ||||||
|  |  | ||||||
|  | ## Changes in Zulip 7.5 | ||||||
|  |  | ||||||
|  | **Feature level 186** | ||||||
|  |  | ||||||
|  | * [`POST /register`](/api/register-queue): `streams` field in the response | ||||||
|  |   now included web-public streams as well. This change was backported from | ||||||
|  |   Zulip 8.0, where it was introduced in feature level 205. | ||||||
|  |  | ||||||
| ## Changes in Zulip 7.0 | ## Changes in Zulip 7.0 | ||||||
|  |  | ||||||
| **Feature level 185** | **Feature level 185** | ||||||
|   | |||||||
| @@ -565,7 +565,9 @@ def fetch_initial_state_data( | |||||||
|         # places. |         # places. | ||||||
|         if user_profile is not None: |         if user_profile is not None: | ||||||
|             state["streams"] = do_get_streams( |             state["streams"] = do_get_streams( | ||||||
|                 user_profile, include_all_active=user_profile.is_realm_admin |                 user_profile, | ||||||
|  |                 include_web_public=True, | ||||||
|  |                 include_all_active=user_profile.is_realm_admin, | ||||||
|             ) |             ) | ||||||
|         else: |         else: | ||||||
|             # TODO: This line isn't used by the web app because it |             # TODO: This line isn't used by the web app because it | ||||||
| @@ -889,7 +891,9 @@ def apply_event( | |||||||
|  |  | ||||||
|                     if "streams" in state: |                     if "streams" in state: | ||||||
|                         state["streams"] = do_get_streams( |                         state["streams"] = do_get_streams( | ||||||
|                             user_profile, include_all_active=user_profile.is_realm_admin |                             user_profile, | ||||||
|  |                             include_web_public=True, | ||||||
|  |                             include_all_active=user_profile.is_realm_admin, | ||||||
|                         ) |                         ) | ||||||
|  |  | ||||||
|                     if state["is_guest"]: |                     if state["is_guest"]: | ||||||
|   | |||||||
| @@ -11559,6 +11559,9 @@ paths: | |||||||
|  |  | ||||||
|                           For organization administrators, this will include all private streams |                           For organization administrators, this will include all private streams | ||||||
|                           in the organization. |                           in the organization. | ||||||
|  |  | ||||||
|  |                           **Changes**: As of Zulip 7.5 (feature level 186), this will include all | ||||||
|  |                           web-public streams in the organization as well. | ||||||
|                       realm_default_streams: |                       realm_default_streams: | ||||||
|                         type: array |                         type: array | ||||||
|                         items: |                         items: | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user