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

fix(navigation): fix navigation to show base form questions #565

Merged
merged 2 commits into from
Oct 11, 2019

Conversation

Yelinz
Copy link
Member

@Yelinz Yelinz commented Sep 30, 2019

Fixes #460

@anehx
Copy link
Contributor

anehx commented Oct 1, 2019

Will this now cause to render a navigation even if there are no subforms? How is it structured - are all subforms indented a bit more or is the main form on the same level? Could you append a screenshot?

@Yelinz
Copy link
Member Author

Yelinz commented Oct 1, 2019

Will this now cause to render a navigation even if there are no subforms?

There is no navigation when there are no subforms

How is it structured - are all subforms indented a bit more or is the main form on the same level?

They are all on the same level

Could you append a screenshot?

Screenshot from 2019-10-01 11-06-56
With subform

Screenshot from 2019-10-01 11-07-30
Without subform

@Yelinz
Copy link
Member Author

Yelinz commented Oct 1, 2019

Now with a divider between main and sub form navigation points
Screenshot from 2019-10-01 12-30-13

Copy link
Contributor

@anehx anehx left a comment

Choose a reason for hiding this comment

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

LGMT

@Yelinz Yelinz merged commit 5f38df0 into projectcaluma:master Oct 11, 2019
@Yelinz Yelinz deleted the fix-navigation branch October 11, 2019 13:46
czosel pushed a commit that referenced this pull request Nov 6, 2019
# [3.0.0](v2.5.0...v3.0.0) (2019-11-06)

### Bug Fixes

* **jexl:** allow falsey values for intersects operator without breaking ([4fbb73f](4fbb73f))
* **navigation:** fix navigation to show base form questions ([#565](#565)) ([5f38df0](5f38df0)), closes [#460](#460)
* **table:** don't re-save existing rows after edit ([e5cd5ef](e5cd5ef))
* **table:** remove row documents on delete ([395e4e4](395e4e4))
* **table:** skip empty dates in table view ([97f5758](97f5758))

### chore

* remove support for ember 3.8 ([093d25b](093d25b))

### Features

* **form:** handle old values of dynamic questions ([c9bbc36](c9bbc36))
* **license:** switch license from MIT to LGPL-3.0-or-later ([b1163eb](b1163eb))

### BREAKING CHANGES

* **license:** Since the LGPL license is more restrictive than MIT,
this is marked as breaking
* **form:** This is a breaking change, since it will only work with
v3.0.0 or later of the caluma service.
* This will remove the support for the second last LTS
version of ember, since we support only the last LTS version which is
3.12 now.
@czosel
Copy link
Contributor

czosel commented Nov 6, 2019

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@czosel czosel added the released label Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Main form questions are not rendered
3 participants