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

Add Bluesky Url to Members Profile #1177

Merged
merged 8 commits into from
Nov 10, 2023

Conversation

Coop56
Copy link
Contributor

@Coop56 Coop56 commented Oct 6, 2023

Adds Bluesky as a social media option on the Members page. I used a cloud icon since there’s not a specific Bluesky icon from font-awesome yet. Open to suggestions on changes if there’s another icon.

@Coop56
Copy link
Contributor Author

Coop56 commented Oct 6, 2023

I see that the CI build is failing. I ran the build and tests locally and it worked. I’ll investigate what’s happening with CI tomorrow.

@ardalis
Copy link
Collaborator

ardalis commented Oct 6, 2023

Looks good. I'll need to do a db change when I deploy and I may not get to that before next week but we'll see.

Thanks!

Copy link
Member

@KyleMcMaster KyleMcMaster left a comment

Choose a reason for hiding this comment

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

LGTM! :shipit:

@@ -17,6 +17,7 @@
private string _initialTwitchUrl = "";
private string _initialYouTubeUrl = "";
private string _initialTwitterUrl = "";
private string _initialBlueskyUrl = "";

Check notice

Code scanning / CodeQL

Missed 'readonly' opportunity Note test

Field '_initialBlueskyUrl' can be 'readonly'.
@sadukie sadukie added the database update Pull requests that contain a database update label Oct 10, 2023
@Coop56 Coop56 marked this pull request as draft October 10, 2023 22:06
@Coop56
Copy link
Contributor Author

Coop56 commented Oct 10, 2023

Converted to Draft just so it doesn't get merged until we're ready.

@Coop56 Coop56 marked this pull request as ready for review October 17, 2023 22:29
@ardalis ardalis merged commit 32b381d into DevBetterCom:main Nov 10, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database update Pull requests that contain a database update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants