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

(refactor) Replace deprecated useSessionUser with useSession #177

Merged
merged 1 commit into from
Apr 20, 2022

Conversation

denniskigen
Copy link
Member

@denniskigen denniskigen commented Apr 20, 2022

Requirements

  • This PR has a title that briefly describes the work done, including the ticket number if there is a ticket.
  • My work conforms to the OpenMRS 3.0 Styleguide.
  • I checked for feature overlap with existing widgets.

Summary

This PR replaces the deprecated useSessionUser function declaration with useSession. This PR does not introduce any functional changes.

Related: openmrs/openmrs-esm-patient-chart#661

@github-actions
Copy link
Contributor

File size impact

Merging chore-react-typings into main impact files as follow:

@openmrs/esm-active-visits-app (no impact)
Files new size
Unmodified (17) 1.39 MB (0 B / +0%) 👻
Total (17) 1.39 MB (0 B / +0%) 👻
@openmrs/esm-outpatient-app (-0%)
Files new size
packages/esm-outpatient-app/dist/608.js 124 kB (-4 B / -0%) ↘️
Unmodified (21) 8.08 MB (0 B / +0%) 👻
Total (22) 8.2 MB (-4 B / -0%) ↘️
@openmrs/esm-patient-list-app (-0%)
Files new size
packages/esm-patient-list-app/dist/414.js 35.7 kB (-4 B / -0.01%) ↘️
packages/esm-patient-list-app/dist/670.js 73.1 kB (-4 B / -0.01%) ↘️
packages/esm-patient-list-app/dist/380.js 190 kB (-12 B / -0.01%) ↘️
Unmodified (24) 1.49 MB (0 B / +0%) 👻
Total (27) 1.79 MB (-20 B / -0%) ↘️
@openmrs/esm-patient-registration-app (no impact)
Files new size
Unmodified (21) 1.73 MB (0 B / +0%) 👻
Total (21) 1.73 MB (0 B / +0%) 👻
@openmrs/esm-patient-search-app (no impact)
Files new size
Unmodified (12) 1.12 MB (0 B / +0%) 👻
Total (12) 1.12 MB (0 B / +0%) 👻
Generated by @jsenv/file-size-impact during Report bundle size#2197133917 on 1055705

@brandones brandones merged commit 8d275ff into main Apr 20, 2022
@brandones brandones deleted the chore-react-typings branch April 20, 2022 20:28
@brandones
Copy link
Contributor

Thanks @denniskigen !

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.

2 participants