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

chore(docs): improve-topics-swagger #2422

Merged
merged 1 commit into from
Jan 4, 2023

Conversation

p-fernandez
Copy link
Contributor

What change does this PR introduce?

Improvements of topics swagger docs.

Why was this change needed?

Chore improvement

Other information (Screenshots)

@p-fernandez p-fernandez self-assigned this Jan 4, 2023
@@ -53,7 +53,7 @@ import { UserSession } from '../shared/framework/user.decorator';
import { AnalyticsService } from '../shared/services/analytics/analytics.service';
import { ANALYTICS_SERVICE } from '../shared/shared.module';

@Controller('topics')
@Controller('/topics')
Copy link
Contributor

@scopsy scopsy Jan 4, 2023

Choose a reason for hiding this comment

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

Out of curiosity, does it not recognize the endpoints without it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I honestly have no idea. But I saw that Subscribers API shows up with that and is the one showing in docs, so I did this guess.

@p-fernandez p-fernandez merged commit 4d6e244 into next Jan 4, 2023
@p-fernandez p-fernandez deleted the chore(docs)-improve-topics-swagger branch January 4, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants