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

Preview strony projektu dla social mediów #41

Merged
merged 4 commits into from
Feb 4, 2024

Conversation

KaNoSki
Copy link
Collaborator

@KaNoSki KaNoSki commented Nov 12, 2023

#38

Strona nie wywala i teoretycznie działa
image

@KaNoSki KaNoSki requested a review from pktiuk November 12, 2023 20:13
@pktiuk
Copy link
Contributor

pktiuk commented Nov 29, 2023

image
Podgląd działa jak trzeba. Do testów użyłem tego dodatku: https://chromewebstore.google.com/detail/social-share-preview/ggnikicjfklimmffbkhknndafpdlabib?pli=1

@pktiuk
Copy link
Contributor

pktiuk commented Nov 29, 2023

Moim zdaniem kod jest dobry.

Jak już będzie gotowy twoim zdaniem to ustaw go jako ready for review

@pktiuk
Copy link
Contributor

pktiuk commented Dec 3, 2023

Closes: #38
(to komenda, która automatycznie zamknie to issue po zmergowaniu tego PRa)

Copy link
Contributor

@pktiuk pktiuk left a comment

Choose a reason for hiding this comment

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

LGTM

@pktiuk
Copy link
Contributor

pktiuk commented Feb 3, 2024

Widzę, że pojawił się konflikt przy mergowaniu.

@KaNoSki KaNoSki marked this pull request as ready for review February 4, 2024 21:01
@pktiuk pktiuk merged commit fa5ed0f into KNR-PW:main Feb 4, 2024
@pktiuk
Copy link
Contributor

pktiuk commented Feb 4, 2024

No i elegancko.
Na ten moment jest już dobrze, więc zmergowałem.

Jakieś dodatkowe poprawki będzie można dodać w oddzielnym pull requeście
obraz

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