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

Bug fixes combo/link/reminders bugs #259

Merged
merged 5 commits into from
Nov 10, 2022

Conversation

xypnox
Copy link
Contributor

@xypnox xypnox commented Nov 10, 2022

  • Fix create new behaviour for combobox elements other than ilinks
  • Fix url deletion request url
  • Fix reminder view link and id to a constant

Conditionally checks the presence of the current namespace in items if
only the command is internal, i.e. [[
@linear
Copy link

linear bot commented Nov 10, 2022

WD-1685 [BUG] Extension combo/link bugs

Describe the bug

  • Combobox shows create new for existing tags as well
  • Delete Links not working

Expected Behaviour

Explain what was supposed to happen

  • Combobox should not show same tags with create new
  • Delete links should work

Screenshots

Slack

https://workduck-workspace.slack.com/archives/C049CHKV8U9/p1667838396217099

https://workduck-workspace.slack.com/archives/C049CHKV8U9/p1667840835155009

Additional Context

NA

@changeset-bot
Copy link

changeset-bot bot commented Nov 10, 2022

🦋 Changeset detected

Latest commit: 2a0763f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
mexit-webapp Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@xypnox xypnox requested review from rpPanda and mukul-mehta and removed request for rpPanda November 10, 2022 13:30
@xypnox xypnox merged commit 5d3b13f into main Nov 10, 2022
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.

2 participants