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

Removed opening hours and added simple-opening-hours #349

Closed
wants to merge 1 commit into from
Closed

Removed opening hours and added simple-opening-hours #349

wants to merge 1 commit into from

Conversation

Kishore-abhimanyu
Copy link

Removed opening hours and added simple-opening-hours

@Kishore-abhimanyu
Copy link
Author

can you please review and merge it?

Copy link
Member

@mojoaxel mojoaxel left a comment

Choose a reason for hiding this comment

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

@Kishore-abhimanyu Thanks for trying to contribute!

  • It is not enough to just replace one library with another, because they are not compatible. You also need to change the usage in the code.
  • The test fail. Please make sure the CI turnes green after you changes.

},
"repository": {
"type": "git",
"url": "https://github.com/wo-ist-markt/wo-ist-markt.github.io"
},
"scripts": {
"scripts": {
Copy link
Member

Choose a reason for hiding this comment

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

This new whitespace should be removed.

Copy link
Member

Choose a reason for hiding this comment

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

@Kishore-abhimanyu Do you need help with this?

Copy link
Member

Choose a reason for hiding this comment

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

@Kishore-abhimanyu Are you still working on this?

Copy link
Member

Choose a reason for hiding this comment

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

@mojoaxel Do you know the state of this pull request?

Copy link
Member

Choose a reason for hiding this comment

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

🛎️ @Kishore-abhimanyu, @mojoaxel Last reminder before closing this.

@johnjohndoe johnjohndoe added the Project setup Dependencies, libraries, frameworks label Oct 24, 2020
@johnjohndoe johnjohndoe added the Abandoned Stale issues or pull requests label Apr 12, 2021
@johnjohndoe johnjohndoe marked this pull request as draft April 12, 2021 12:47
@johnjohndoe johnjohndoe self-assigned this May 29, 2021
@johnjohndoe
Copy link
Member

I am now closing this issue due to inactivity over the last months. If anyone is interested in the topic we can re-open it at any time. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Abandoned Stale issues or pull requests Project setup Dependencies, libraries, frameworks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants