Skip to content

Conversation

@manuwei
Copy link
Contributor

@manuwei manuwei commented Mar 25, 2021

Allow pattern authors to reference other patterns within the description of a pattern via hyperlinks:

  • when creating a new pattern (via a new button that was added to the the markdown editor)
  • when editing a pattern (via a new select field in the editor-dialog)

This fixes #81

Manuela Weigold added 8 commits March 24, 2021 14:43
…kdown-pattern-references

# Conflicts:
#	src/app/core/core.module.ts
#	src/app/core/default-pl-renderer/default-pl-renderer.component.ts
#	src/app/pattern-language-management/create-pattern/create-pattern.component.ts
pattern language needs to be loaded first, because we need the pattern language object to load the pattern afterwards
@manuwei manuwei requested a review from mbeisel March 25, 2021 14:24
@ghost
Copy link

ghost commented Mar 25, 2021

Congratulations 🎉. DeepCode analyzed your code in 2.508 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@mbeisel
Copy link
Contributor

mbeisel commented Apr 30, 2021

Looks good!
I would just do some minor change regarding the look.

  1. Add some spacing in between the dropdown and the add button
  2. Test if it would look better, to put the dropdown to the right side of the title to save some space

grafik

@ghost
Copy link

ghost commented May 25, 2021

Congratulations 🎉. DeepCode analyzed your code in 2.508 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@ghost
Copy link

ghost commented May 25, 2021

DeepCode failed to analyze this pull request

Something went wrong despite trying multiple times, sorry about that.
Please comment this pull request with "Retry DeepCode" to manually retry, or contact us so that a human can look into the issue.

@ghost
Copy link

ghost commented May 25, 2021

Congratulations 🎉. DeepCode analyzed your code in 2.508 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@ghost
Copy link

ghost commented May 25, 2021

DeepCode failed to analyze this pull request

Something went wrong despite trying multiple times, sorry about that.
Please comment this pull request with "Retry DeepCode" to manually retry, or contact us so that a human can look into the issue.

@ghost
Copy link

ghost commented May 25, 2021

Congratulations 🎉. DeepCode analyzed your code in 2.508 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@ghost
Copy link

ghost commented May 25, 2021

DeepCode failed to analyze this pull request

Something went wrong despite trying multiple times, sorry about that.
Please comment this pull request with "Retry DeepCode" to manually retry, or contact us so that a human can look into the issue.

@manuwei
Copy link
Contributor Author

manuwei commented May 25, 2021

Retry DeepCode

@ghost
Copy link

ghost commented May 25, 2021

Congratulations 🎉. DeepCode analyzed your code in 0.598 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@ghost
Copy link

ghost commented May 25, 2021

Congratulations 🎉. DeepCode analyzed your code in 4.799 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

👉 The DeepCode service and API will be deprecated in August, 2021. Here is the information how to migrate. Thank you for using DeepCode 🙏 ❤️ !

If you are using our plugins, you might be interested in their successors: Snyk's JetBrains plugin and Snyk's VS Code plugin.

Manuela Weigold added 3 commits May 26, 2021 09:04
…list-4.16.6' into feature/markdown-pattern-references
…kdown-pattern-references

# Conflicts:
#	src/app/core/component/cardrenderer/card-renderer.component.ts
#	src/app/core/core.module.ts
#	src/app/pattern-language-management/create-pattern/create-pattern.component.ts
#	src/app/pattern-language-management/pattern-language-management/pattern-language-management.component.ts
#	src/app/pattern-view-management/pattern-view-management/pattern-view-management.component.ts
#	src/app/pattern-view-management/pattern-view-renderer/pattern-view-renderer.component.ts
@manuwei
Copy link
Contributor Author

manuwei commented Jun 7, 2021

@mbeisel could you recheck the changes? besides the css changes, it's mostly whitespace :)

@mbeisel mbeisel merged commit 75d31c8 into master Jun 9, 2021
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.

Add functionality to create links to other patterns

3 participants