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

[FIX][ENTERPRISE] Leading slashes in Engagement Dashboard API requests #24142

Merged
merged 1 commit into from Jan 12, 2022

Conversation

matheusbsilva137
Copy link
Contributor

Proposed changes (including videos or screenshots)

  • Remove trailing slashes from Engagement Dashboard API requests;

Issue(s)

Steps to test or reproduce

Go to Administration > Engagement Dashboard. Open DevTools > Network and check one the API requests related to the ED:
Before: http://localhost:3000/api//v1/engagement-dashboard/users/...
After: http://localhost:3000/api/v1/engagement-dashboard/users/...

Further comments

@matheusbsilva137 matheusbsilva137 changed the title [FIX][ENTERPRISE] Trailing slashs in Engagement Dashboard API requests [FIX][ENTERPRISE] Leading slashes in Engagement Dashboard API requests Jan 12, 2022
@matheusbsilva137 matheusbsilva137 merged commit d43f538 into develop Jan 12, 2022
@matheusbsilva137 matheusbsilva137 deleted the fix/api-requests-engagement-dashboard branch January 12, 2022 13:37
@matheusbsilva137 matheusbsilva137 added this to the 4.3.2 milestone Jan 12, 2022
gabriellsh added a commit that referenced this pull request Jan 17, 2022
…ove/setup-wizard

* 'develop' of github.com:RocketChat/Rocket.Chat: (176 commits)
  [IMPROVE] Admin page header buttons consistency (#24168)
  i18n: Language update from LingoHub 🤖 on 2022-01-17Z (#24193)
  [FIX] Integration section crashing opening in My Account (#24068)
  [IMPROVE] Rewrite roomNotFound to React Component (#24044)
  Regression: Enable custom emoji on admin custom status page (#24186)
  Chore: Update Meteor to 2.5.3 (#24075)
  [NEW] Apple Login (#24060)
  Chore: Update Apps-Engine to 1.29.2 (#24171)
  feat: enabling emoji on custom status (#24170)
  [FIX] App Framework Enable hanging indefinitely (#24158)
  [FIX] CSV Importer failing to import users (#24090)
  Fix Engagement Dashboard API requests (#24142)
  Language update from LingoHub 🤖 (#24127)
  Chore: Migrate useOutsideClick to fuselage-hooks (#24133)
  Revert "Use fibers to store context"
  Use fibers to store context
  Chore: Include REG_TOKEN in docker-compose (#24123)
  [FIX] Custom Emoji Image preview #24117
  [IMPROVE] Added a Reset Button in the Account Profile Page (#24078)
  Revert: "[IMPROVE] Throw 404 error in invalid endpoints" (#24118)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Jan 19, 2022
gabriellsh added a commit that referenced this pull request Jan 19, 2022
…hat into improve/setup-wizard

* 'improve/setup-wizard' of github.com:RocketChat/Rocket.Chat:
  Regression: Fix handling of http requests in apps bridge (#24211)
  Bump version to 4.3.2
  [FIX] Integration section crashing opening in My Account (#24068)
  Chore: Update Apps-Engine to 1.29.2 (#24171)
  [FIX] App Framework Enable hanging indefinitely (#24158)
  [FIX] CSV Importer failing to import users (#24090)
  Fix Engagement Dashboard API requests (#24142)
  [FIX] MAU when using micro services (#24204)
  [IMPROVE] Limit recent emojis to 27 (#24210)
  Bump version to 4.3.1
  Bump Fuselage packages
  [FIX][APPS] Action buttons not removed when app is disabled or uninstalled (#24107)
  [FIX][APPS] Prevents emails from being sent when apps framework is disabled (#24105)
  [FIX] Ensure Firefox 91 ESR support (#24096)
  Chore: Update Livechat (#24091)
  [FIX] Omnichannel enabled setting not working when creating rooms (#24067)
  [FIX] Enter not working on modal's multi-line input (#23981)
  [FIX] Omnichannel Current chats pagination not working (#24039)
@sampaiodiego sampaiodiego mentioned this pull request Jan 29, 2022
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.

None yet

2 participants