Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Create OcContextualHelper component #2064

Merged
merged 2 commits into from
Apr 20, 2022
Merged

Create OcContextualHelper component #2064

merged 2 commits into from
Apr 20, 2022

Conversation

lookacat
Copy link
Contributor

@lookacat lookacat commented Apr 6, 2022

Description

See owncloud/web#6590

Related Issue

Screenshots

Screenshot (10)

Types of changes

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

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation added/updated

@update-docs
Copy link

update-docs bot commented Apr 6, 2022

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@lookacat lookacat changed the title [WIP] Implement basic component structure [WIP] Create OcContextualHelper Apr 7, 2022
@lookacat
Copy link
Contributor Author

lookacat commented Apr 7, 2022

For clarification: Of couse slots would be better in most cases but the idea here is to give all information by props so we can create json files with the content of each helper in web. This would mean almost no markup in web, if we used slots it would create markup that is not relevant for web.

@lookacat lookacat force-pushed the add-contextual-helper branch 2 times, most recently from 712ddc2 to d704db8 Compare April 7, 2022 14:20
@lookacat lookacat changed the title [WIP] Create OcContextualHelper Create OcContextualHelper component Apr 8, 2022
@lookacat lookacat marked this pull request as ready for review April 8, 2022 09:24
Copy link
Contributor

@pascalwengerter pascalwengerter left a comment

Choose a reason for hiding this comment

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

A bunch of small improvement ideas/questions, like it a lot so far already!

Copy link
Contributor

@pascalwengerter pascalwengerter left a comment

Choose a reason for hiding this comment

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

Sorry, more findings 🐺

Copy link
Contributor

@pascalwengerter pascalwengerter left a comment

Choose a reason for hiding this comment

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

😬

@lookacat
Copy link
Contributor Author

done :)

@lookacat lookacat force-pushed the add-contextual-helper branch 2 times, most recently from a596785 to ec5f9ae Compare April 20, 2022 10:06
Copy link
Contributor

@pascalwengerter pascalwengerter left a comment

Choose a reason for hiding this comment

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

LGTM 🖖🏽

@sonarcloud
Copy link

sonarcloud bot commented Apr 20, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

91.7% 91.7% Coverage
0.0% 0.0% Duplication

@lookacat lookacat merged commit 633f5b9 into master Apr 20, 2022
@delete-merged-branch delete-merged-branch bot deleted the add-contextual-helper branch April 20, 2022 10:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants