-
-
Notifications
You must be signed in to change notification settings - Fork 230
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: connect more user profile settings to the API #826
Conversation
β Deploy Preview for oss-insights ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
β Deploy Preview for design-insights ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.
Issue Reference
In order to be considered for merging, the pull request description must refer to a specific issue number. This is described in our Contributing Guide.
This check is looking for a phrase similar to: "Fixes #XYZ" or "Resolves #XYZ" where XYZ is the issue number that this PR is meant to address.
I am seeing weirdness when selecting a timezone. I can't select Mountain Time, it defaults to La Paz. I chose mountain time, but it shows La Paz cc @OgDev-01 |
@bdougie should be fixed now. I thought we could just use the offset, but its not unique so I switched it back to the string value |
components/molecules/ContributorProfileInfo/contributor-profile-info.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
πππππππππππ
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM πππ
## [1.25.0-beta.14](v1.25.0-beta.13...v1.25.0-beta.14) (2023-02-01) ### π Bug Fixes * connect more user profile settings to the API ([#826](#826)) ([01b8bc0](01b8bc0))
π This PR is included in version 1.25.0-beta.14 π The release is available on GitHub release Your semantic-release bot π¦π |
## [1.26.0-beta.1](v1.25.0...v1.26.0-beta.1) (2023-02-02) ### π Features * add profile badge to connected user profile ([#824](#824)) ([e3a1fcf](e3a1fcf)), closes [#821](#821) [#820](#820) * add user location and bio details to profile page ([#797](#797)) ([a301874](a301874)) * implement profile settings feature ([#808](#808)) ([4a8d706](4a8d706)) ### π Bug Fixes * connect more user profile settings to the API ([#826](#826)) ([01b8bc0](01b8bc0))
π This PR is included in version 1.26.0-beta.1 π The release is available on GitHub release Your semantic-release bot π¦π |
## [1.26.0-beta.1](v1.25.0...v1.26.0-beta.1) (2023-02-02) ### π Features * add profile badge to connected user profile ([#824](#824)) ([e3a1fcf](e3a1fcf)), closes [#821](#821) [#820](#820) * add user location and bio details to profile page ([#797](#797)) ([a301874](a301874)) * implement profile settings feature ([#808](#808)) ([4a8d706](4a8d706)) ### π Bug Fixes * connect more user profile settings to the API ([#826](#826)) ([01b8bc0](01b8bc0)) ### π€ Build System * correct dep versioning ([42a1b0a](42a1b0a)) * update some dependencies ([f522f5e](f522f5e))
π This PR is included in version 1.26.0-beta.1 π The release is available on GitHub release Your semantic-release bot π¦π |
## [1.26.0](v1.25.0...v1.26.0) (2023-02-02) ### π Features * add profile badge to connected user profile ([#824](#824)) ([e3a1fcf](e3a1fcf)), closes [#821](#821) [#820](#820) * add user location and bio details to profile page ([#797](#797)) ([a301874](a301874)) * implement profile settings feature ([#808](#808)) ([4a8d706](4a8d706)) ### π Bug Fixes * connect more user profile settings to the API ([#826](#826)) ([01b8bc0](01b8bc0)) ### π€ Build System * correct dep versioning ([42a1b0a](42a1b0a)) * update some dependencies ([f522f5e](f522f5e))
π This PR is included in version 1.26.0 π The release is available on GitHub release Your semantic-release bot π¦π |
* origin/main: (37 commits) chore(minor): release 1.26.0 [skip ci] chore(minor): release 1.26.0-beta.1 on beta channel [skip ci] build: update some dependencies build: correct dep versioning chore(patch): release 1.25.0-beta.14 on beta channel [skip ci] fix: connect more user profile settings to the API (#826) chore(minor): release 1.25.0-beta.13 on beta channel [skip ci] feat: add profile badge to connected user profile (#824) chore(minor): release 1.25.0-beta.12 on beta channel [skip ci] feat: add user location and bio details to profile page (#797) chore(minor): release 1.25.0-beta.11 on beta channel [skip ci] feat: implement profile settings feature (#808) chore(minor): release 1.25.0 [skip ci] (#818) chore(minor): release 1.25.0 [skip ci] v1.25.0-beta.10 -> production (#815) chore(minor): release 1.25.0-beta.10 on beta channel [skip ci] feat: set insight repo limit based on role (#813) chore(patch): release 1.25.0-beta.9 on beta channel [skip ci] fix: use correct avatar URL for caching (#800) chore(minor): release 1.25.0-beta.8 on beta channel [skip ci] ...
## [1.25.0-beta.14](open-sauced/app@v1.25.0-beta.13...v1.25.0-beta.14) (2023-02-01) ### π Bug Fixes * connect more user profile settings to the API ([#826](open-sauced/app#826)) ([01b8bc0](open-sauced/app@01b8bc0))
## [1.26.0-beta.1](open-sauced/app@v1.25.0...v1.26.0-beta.1) (2023-02-02) ### π Features * add profile badge to connected user profile ([#824](open-sauced/app#824)) ([e3a1fcf](open-sauced/app@e3a1fcf)), closes [#821](open-sauced/app#821) [#820](open-sauced/app#820) * add user location and bio details to profile page ([#797](open-sauced/app#797)) ([a301874](open-sauced/app@a301874)) * implement profile settings feature ([#808](open-sauced/app#808)) ([4a8d706](open-sauced/app@4a8d706)) ### π Bug Fixes * connect more user profile settings to the API ([#826](open-sauced/app#826)) ([01b8bc0](open-sauced/app@01b8bc0)) ### π€ Build System * correct dep versioning ([42a1b0a](open-sauced/app@42a1b0a)) * update some dependencies ([f522f5e](open-sauced/app@f522f5e))
## [1.26.0](open-sauced/app@v1.25.0...v1.26.0) (2023-02-02) ### π Features * add profile badge to connected user profile ([#824](open-sauced/app#824)) ([e3a1fcf](open-sauced/app@e3a1fcf)), closes [#821](open-sauced/app#821) [#820](open-sauced/app#820) * add user location and bio details to profile page ([#797](open-sauced/app#797)) ([a301874](open-sauced/app@a301874)) * implement profile settings feature ([#808](open-sauced/app#808)) ([4a8d706](open-sauced/app@4a8d706)) ### π Bug Fixes * connect more user profile settings to the API ([#826](open-sauced/app#826)) ([01b8bc0](open-sauced/app@01b8bc0)) ### π€ Build System * correct dep versioning ([42a1b0a](open-sauced/app@42a1b0a)) * update some dependencies ([f522f5e](open-sauced/app@f522f5e))
What type of PR is this? (check all applicable)
Description
This PR:
Display Local Time
andTimezone
to the API for updatesRelated Tickets & Documents
Mobile & Desktop Screenshots/Recordings
Added tests?
Added to documentation?
[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?