Skip to content

Added Code of Conduct and Cookie Policy Page #207

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

Merged
merged 8 commits into from
May 24, 2024

Conversation

Hemashree21
Copy link
Contributor

@Hemashree21 Hemashree21 commented May 23, 2024

Resolves issue #201. I have added the Code of Conduct and Cookie Policy page, both are in the same format as the Privacy Policy page.

image

image

@ajay-dhangar please check once

Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
@Hemashree21 Hemashree21 requested a review from ajay-dhangar as a code owner May 23, 2024 17:41
Copy link

vercel bot commented May 23, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
codeharborhub-github-io ✅ Ready (Inspect) Visit Preview 💬 1 unresolved May 24, 2024 3:26am

Copy link
Contributor

deepsource-io bot commented May 23, 2024

Here's the code health analysis summary for commits 4c097d2..812b858. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
@Hemashree21
Copy link
Contributor Author

@ajay-dhangar The Deepsource: JavaScript is showing deepnesting error so the check is not passing. If I have to resolve that I have to rewrite the complete code. Is it compulsory to pass all the checks here and will it create some conflict if not resolved? If I need to resolve it I will do it tomorrow.

Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
@ajay-dhangar ajay-dhangar added documentation Improvements or additions to documentation GSSOC'24 GirlScript Summer of Code | Contributor level2 GirlScript Summer of Code | Contributor's Levels gssoc GirlScript Summer of Code | Contributor labels May 23, 2024
@ajay-dhangar ajay-dhangar linked an issue May 23, 2024 that may be closed by this pull request
Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

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

Changes something your aim and you can follow our suggestions for some Idea

padding: 20px;
border-radius: 5px;
box-shadow: 0 0 10px 0 rgba(0, 0, 0, 0.1);
margin-top: 20px;
Copy link
Member

Choose a reason for hiding this comment

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

remove margin-top: 20px; , margin-bottom: 20px; and update margin: 20px auto;

* Cookie Section
**/

const CookieSection = ({ title, children }) => (
Copy link
Member

Choose a reason for hiding this comment

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

in the case of tsx you need to define react function for example

bad practices

const CookieSection = ({ title, children }) => (
<section>
    <SubTitle>{title}</SubTitle>
    {children}
  </section>
);

good practices

const CookieSection: React.FC = ({ title, children }) => {
  return (
     <section>
     <SubTitle>{title}</SubTitle>
          {children}
      </section>
    );
};

* Cookie Policy
**/

const Cookiepolicy = () => {
Copy link
Member

Choose a reason for hiding this comment

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

also define const Cookiepolicy = () => { to const Cookiepolicy: React.FC = () => {

Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
Signed-off-by: Hemashree M S <hemashree21.msh@gmail.com>
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@Hemashree21
Copy link
Contributor Author

@ajay-dhangar I resolved all issues and made the changes as suggested

Copy link
Member

@ajay-dhangar ajay-dhangar left a comment

Choose a reason for hiding this comment

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

Done

@ajay-dhangar ajay-dhangar merged commit 989c6ae into codeharborhub:main May 24, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation gssoc GirlScript Summer of Code | Contributor GSSOC'24 GirlScript Summer of Code | Contributor level2 GirlScript Summer of Code | Contributor's Levels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Adding Code of Conduct and Cookie Policy Page
2 participants