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 heart aa1 to implement 1 #49

Merged
merged 31 commits into from
Jun 7, 2020
Merged

Add heart aa1 to implement 1 #49

merged 31 commits into from
Jun 7, 2020

Conversation

pinterid
Copy link
Member

@pinterid pinterid commented Jun 6, 2020

Checklist:

  • Have you added an explanation of what your changes do and why you'd like them to be included?
  • Have you updated or added documentation for the change?
  • Have you tested your changes with successful results?

Type of Changes:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation (non-breaking change which adds documentation)
  • Breaking change (fix or feature that would cause existing functionality to change)

What is the current behavior? (link to any open issues here)

What is the new behavior (if this is a feature change)?

Other information:

  • Ref: 🐍

schettn added 23 commits June 6, 2020 21:25
A utility for themes has been added.
Implemented for export them for
easy access from parent components.
Implemented for export them for
easy access from parent components.
Now the modals are imported directly from
the models folder.
Added some utilities for base and variables.
The navbar and footer are displayed now.
Methods for authentications has been
added.
js-sha256, obelisk.js, react-dropzone
and tippy.js has been added.
A progress bar has been added.
The homepage is displayed now.
Now when pressing the login button
in the navbar, it will be redirected to
the root location.
Caching methods and data handling for
intel data have been implemented.
Registration is now available. This includes
a oAuth and gitlab connection.
The companypage is displayed now.
Starts the page on top when reloaded 
or redirected.
The profilepage is displayed now.
A settings modal has been added and
implemented.
A invalid variable type has changed.
Now it is possible to upload and
delete talks.
The site title has been changed to SNEK.
The talkspage is displayed now.
Now only the globalFunctions and
globalState are passed as props.
Code quality has been improved
@pinterid pinterid added the enhancement New feature or request label Jun 6, 2020
@pinterid pinterid added this to the SNEK Version 0.1.0 milestone Jun 6, 2020
@pinterid pinterid self-assigned this Jun 6, 2020
Copy link
Member

@schettn schettn left a comment

Choose a reason for hiding this comment

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

I left some suggestions

public/manifest.json Outdated Show resolved Hide resolved
src/components/pages/TalkPage/index.jsx Outdated Show resolved Hide resolved
src/App.js Outdated Show resolved Hide resolved
Copy link
Member Author

@pinterid pinterid left a comment

Choose a reason for hiding this comment

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

Some changes have to be made.

src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
src/App.js Show resolved Hide resolved
schettn and others added 2 commits June 7, 2020 01:16
Suggestions from @pinterid and @schettn are
applied now.

Co-authored-by: Pinterics David <55298934+pinterid@users.noreply.github.com>
The mount condition has been refined
and simplified.
@schettn schettn self-requested a review June 6, 2020 23:18
Copy link
Member

@schettn schettn left a comment

Choose a reason for hiding this comment

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

LGTM

The code quality was improved due to request by Codacy.
The code quality was improved due to a request by Codacy.
The code quality was improved due to a request by Codacy.
Copy link
Member

@kleberbaum kleberbaum left a comment

Choose a reason for hiding this comment

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

Should be ok to merge after a few changes.

public/index.html Outdated Show resolved Hide resolved
pinterid and others added 2 commits June 7, 2020 02:38
The code quality was improved due to a request by @kleberbaum.

Co-authored-by: Florian Kleber <kleberbaum@erebos.xyz>
The site title was changed due to a request by @kleberbaum.

Co-authored-by: Florian Kleber <kleberbaum@erebos.xyz>
@pinterid pinterid requested a review from kleberbaum June 7, 2020 00:39
Copy link
Member Author

@pinterid pinterid left a comment

Choose a reason for hiding this comment

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

Looks good to merge.

Copy link
Member

@kleberbaum kleberbaum left a comment

Choose a reason for hiding this comment

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

Should be ok to merge.

@schettn schettn merged commit 1e5d8b5 into implement-1 Jun 7, 2020
@pinterid pinterid mentioned this pull request Jun 7, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants