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

Use URLSearchParams instead of transitive dependency querystring #13138

Merged
merged 9 commits into from
Jul 16, 2021

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Apr 12, 2020

We really need to stop using transitive dependencies like this. It could cause our app to break if we remove an unrelated dep or if they change their deps.

For #7842
Requires matrix-org/matrix-react-sdk#4399

Signed-off-by: Michael Telatynski <[email protected]>
Signed-off-by: Michael Telatynski <[email protected]>
Copy link

@rohith5112 rohith5112 left a comment

Choose a reason for hiding this comment

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

i approved the changes

…uy/querystring

� Conflicts:
�	package.json
�	src/@types/global.d.ts
�	src/vector/app.tsx
�	src/vector/jitsi/index.ts
�	src/vector/platform/WebPlatform.ts
�	yarn.lock
@t3chguy t3chguy marked this pull request as ready for review July 16, 2021 12:41
@t3chguy t3chguy requested a review from a team July 16, 2021 12:46
@t3chguy t3chguy added the T-Task Tasks for the team like planning label Jul 16, 2021
@t3chguy t3chguy merged commit ae2e3e8 into develop Jul 16, 2021
@t3chguy t3chguy deleted the t3chguy/querystring branch July 16, 2021 18:57
williamkray pushed a commit to williamkray/element-web that referenced this pull request Jul 28, 2021
[Full Changelog](element-hq/element-web@v1.7.33...v1.7.34-rc.1)
* Translations update from Weblate
[\element-hq#18260](element-hq#18260)
* Update ESLint config
[\element-hq#18236](element-hq#18236)
* Update matrix-org-eslint-plugin and tighten max warning limit
[\element-hq#18207](element-hq#18207)
* Update webpack path for new voice message worker location
[\element-hq#18187](element-hq#18187)
* yarn upgrade
[\element-hq#18150](element-hq#18150)
* Conform to new typescript eslint rules
[\element-hq#18065](element-hq#18065)
* Add the Matrix Space to the translations guide
[\element-hq#18102](element-hq#18102)
* Update eslint-plugin-matrix-org
[\element-hq#18058](element-hq#18058)
* Fix browser history getting stuck looping back to the same room
[\element-hq#18053](element-hq#18053)
* Use URLSearchParams instead of transitive dependency `querystring`
[\element-hq#13138](element-hq#13138)
* Update PR template for new changelog generation
[\element-hq#18003](element-hq#18003)
* Fix space shortcuts on layouts with non-English keys in the places of
numbers
[\element-hq#17780](element-hq#17780)
* Add comment around feature_spaces flag usage
[\element-hq#18006](element-hq#18006)
* Standardise casing of identity server
[\element-hq#17980](element-hq#17980)
* Convert CONTRIBUTING to markdown
[\element-hq#18002](element-hq#18002)
* Ignore workers for jest tests because it disagrees with worker-loader
[\element-hq#17999](element-hq#17999)
* Use webpack worker-loader instead of homegrown hack
[\element-hq#17963](element-hq#17963)
* Fix types to match Typescripted IndexedDB store
[\element-hq#17962](element-hq#17962)
* Improve and consolidate typing
[\element-hq#17931](element-hq#17931)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Tasks for the team like planning
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants